bugPingus - Bugs: bug #33745, Miner and basher behavior is...

 
 

bug #33745: Miner and basher behavior is flawed (I believe) w/patch

Submitter:  Anders Skogheim Liane <maneatingduck>
Submitted:  Sun 10 Jul 2011 07:32:19 PM UTC
   
 
Category:  Game Engine Severity:  3 - Normal
Item Group:  Incorrect Behaviour Status:  Fixed
Privacy:  Public Assigned to:  None
Open/Closed:  Closed Release:  SVN
Operating System:  All Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Sun 10 Jul 2011 07:32:19 PM UTC, original submission:  

I noticed that a Miner digging through a ceiling would drop down while still leaving a two-"pixel" thick amount of ground. This means that other pingus won't fall down through the same hole, which is inconsistent at best. It also makes you unable to save all pingus on the Halloween 2007 level "Locked up by the witch", and is inconvenient in numerous other levels.

Additionally the basher would dig two pixels above the level of his/her feet, which just feels wrong and isn't consistent with the original Lemmings if I remember correctly.

I've included a patch which I believe corrects this behaviour by adjusting the y location of ground removed, but I don't know nowhere enough about the game engine to know whether this breaks anything else in the game. Review by an experienced developer is probably advisable. I'm also not an experienced C++ programmer, so go easy on me :)

Anders Skogheim Liane <maneatingduck>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by grumbel (Updated the item)
  • -email is unavailable- added by maneatingduck (Submitted the item)
  • -email is unavailable- added by maneatingduck (Og her er grave-fiksen :))
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2011-09-05 grumbel StatusNone Fixed
        Open/ClosedOpen Closed
    2011-07-10 maneatingduck Attached File- Added pingus_miner_basher.patch, #23636
        Carbon-Copy- Added -email is unavailable-

    Back to the top

    Powered by Savane 3.13-4b48.
    Corresponding source code