 |
This is caused by a mysterious -28 offset in this line: https://github.com/coelckers/gzdoom/blob/master/src/r_plane.cpp#L1382 If it is removed, this bug goes away. I tried launching Doom and Doom 2 with this offset removed and the top of the sky texture then matches perfectly the top of the screen, which I assume is how it worked in original DOS Doom. Anyone know why such an offset is needed? |
|
|
That line of code, including the comment above for 'careful positioning' originates directly from MBF so this will require some careful (heh) comparison to ensure that changing it won't break compatibility with Boom maps. |
|
|
I can see that in r_sky.cpp that constant is used again, but only applied if the texture is exactly between 128 and 199 in height. One possibility is that the constant was moved at some point and then mistakenly applied to the sky linedef case. Or it might be that it should also only be applied for the 128-199 range here. If someone could check if some Boom maps (especially with sky textures less tall than 200 pixels) broke or not in the next nightly build of QZDoom, then that'd be really nice. (yes, I'm too lazy to check it myself ;)) |
|
|
Boom never had tall skies, so the bug here is that an adjustment needed for 128 pixel skies is applied to all ones. This should use the same positioning logic as all the rest, of course. |
|
|
Using q1.3pre-194-g1b750c1 for QZDoom, the issue has been fixed. Thanks for finally fixing this! |
|
|
You're welcome, Hellser. Going to mark this one as resolved then. |
- Anonymous
