A_JumpIf condition not working correctly with ScaleX (4.8.2)

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.
kipo
Posts: 125
Joined: Mon Oct 07, 2019 1:31 am

A_JumpIf condition not working correctly with ScaleX (4.8.2)

Post by kipo »

(With GZDoom g4.8.2)

Attached is a test file of a zombieman that should increase in size, fire, and shrink again.
In past versions this works fine, but in the latest release the zombieman just freezes without firing.

This was reported to me since it breaks some parts of my mod https://www.moddb.com/mods/dn3doom
You do not have the required permissions to view the files attached to this post.
User avatar
Player701
 
 
Posts: 1575
Joined: Wed May 13, 2009 3:15 am
Graphics Processor: nVidia with Vulkan support

Re: A_JumpIf condition not working correctly with ScaleX (4.

Post by Player701 »

Bug confirmed for 4.8.1 and 4.8.2 as well as the latest master (g4.9pre-85-ga4aca9bd9), missing in 4.7.0. Bisecting reveals this commit to be the cause of the issue, ref. PR #1632. The bug is likely not JIT-related: setting +vm_jit 0 does not affect the current behavior.
User avatar
Player701
 
 
Posts: 1575
Joined: Wed May 13, 2009 3:15 am
Graphics Processor: nVidia with Vulkan support

Re: A_JumpIf condition not working correctly with ScaleX (4.

Post by Player701 »

It appears that the fix was to change the scale values from doubles to floats again, but wasn't it the intent of PR #1632 to make them floats in the first place?...
User avatar
Graf Zahl
Lead GZDoom+Raze Developer
Lead GZDoom+Raze Developer
Posts: 48658
Joined: Sat Jul 19, 2003 10:19 am
Location: Germany

Re: A_JumpIf condition not working correctly with ScaleX (4.

Post by Graf Zahl »

This reverts them to doubles. Since all floating point math is done in doubles, there were imprecisions when comparing values with the reduced precision.
User avatar
Player701
 
 
Posts: 1575
Joined: Wed May 13, 2009 3:15 am
Graphics Processor: nVidia with Vulkan support

Re: A_JumpIf condition not working correctly with ScaleX (4.

Post by Player701 »

Ah, so that's what the problem was - precision issues. I see now, thanks for the explanation.

Return to “Closed Bugs [GZDoom]”