Jump to content

Static analyzer found a small issue in ZDoom code inside EE


Recommended Posts

As you know EE uses ZDoom's 3D object clipping code, because it's pretty much the only 3D object clipping code ever written for DOOM that *just works*. But MP2E's static analyzer did turn up this, in EE's P_CheckPosition3D, which is curious:

612   // Nothing is blocking us, but this actor potentially could
613   // if there is something else to step on.
614   fakedblocker = clip.BlockingMobj;
      
      Value stored to 'fakedblocker' is never read
I wonder what this is supposed to do, and if it is indicative of a problem that may have already been fixed in ZDoom.

Share this post


Link to post

That code is as dead in ZDoom as it is in Eternity. It seems to be a leftover of something that either never got finished or was removed before any source containing it was released. Even the oldest source version I have containing this line doesn't do anything with it.

Share this post


Link to post

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...