Integer Overflow and incompatibility with Boom's scrolls

Bugs that have been investigated and resolved somehow.

Moderator: GZDoom Developers

Forum rules
Please don't bump threads here if you have a problem - it will often be forgotten about if you do. Instead, make a new thread here.
Post Reply
entryway
Posts: 59
Joined: Tue Aug 30, 2005 1:04 pm

Integer Overflow and incompatibility with Boom's scrolls

Post by entryway »

1. Even with «boom (strict)» compatibility position of the blue armor after teleporting is not the same as in Boom (on the center). Shouldn't it?
2. After opening the door on the test level from the attach ZDoom crashes within ~2 minutes with Integer Overflow message. GZDoom does not.
Attachments
crashtest.zip
(34.19 KiB) Downloaded 91 times
User avatar
Graf Zahl
Lead GZDoom+Raze Developer
Lead GZDoom+Raze Developer
Posts: 49067
Joined: Sat Jul 19, 2003 10:19 am
Location: Germany

Re: Integer Overflow and incompatibility with Boom's scrolls

Post by Graf Zahl »

I couldn't reproduce the crash - integer overflow notrmally can't happen unless there's a problem with the CPU - but the position issue is interesting:

It does get placed correctly, there's just a small issue with the order in which things get done:

- The move is too fast so it gets split in two.
- The first half of the move performs the teleport
- The second half of the move never realizes that the situation has changed and still does its thing.

Of course the direction it tries to move the armor in is wrong now. Since cause and effect are quite separated in the code this is going to be tricky...
The entire movement splitting in P_XYMovement seems to run bad if a move is blocked the first time around but in most cases the results are barely noticable.
User avatar
Graf Zahl
Lead GZDoom+Raze Developer
Lead GZDoom+Raze Developer
Posts: 49067
Joined: Sat Jul 19, 2003 10:19 am
Location: Germany

Re: Integer Overflow and incompatibility with Boom's scrolls

Post by Graf Zahl »

fixed, since this also affected angle-changing portals it just became far more important to be addressed.
Post Reply

Return to “Closed Bugs [GZDoom]”