-
Notifications
You must be signed in to change notification settings - Fork 346
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
r_aspectRatio -1 crashes the engine when trying to connect to a server #70
Comments
I'll try to reproduce that later. On Tue, May 14, 2013 at 11:39 PM, motorsep notifications@github.com wrote:
|
windowed mode, 1280x720 |
This seems to be a known bug, see dhewm3/neo/game/Game_local.cpp Line 2428 in fe0ee16
(that comment is from the originally released source) If I find some time and motivation I'll try to investigate it further |
Thanks |
Please replace idGameLocal::CalcFov() from neo/game/Game_local.cpp with https://gist.github.com/DanielGibson/5595590 and post the output. |
Any news? |
No, not yet.. I am not sure whether it was game code causing it or something else. Still under investigation. |
You can check my horrible solution here: |
did you try https://gist.github.com/DanielGibson/5595590 to get some more infos? I haven't looked into this issue in years because the original reporter failed to deliver more information.. |
I can test it, no problem. my code is just dhewm3's fork with minor changes, so it is still a valid environment. ratio_x = 0.000000, ratio_y = 0.000000drapded: /home/me/drap/dhewm3/trunk/neo/game/Game_local.cpp:2424: void idGameLocal::CalcFov(float, float&, float&) const: Assertion `ratio_x > 0 && ratio_y > 0' failed. |
I couldn't reproduce this.. but I added a workaround for the case that ratio_x or ratio_y is 0 anyway. |
Was trying to test multiplayer again yesterday, and got this:
assertion fail!
Game_local.cpp
Line 2767
Expression ( fov_x > 0 ) && ( fov_y > 0)
Windows 7 Pro 64bit (I recall the other guy had Linux)
setting r_aspectRatio to any other available value solves the crash, but doesn't solve having autodetection of aspect ratio working.
The text was updated successfully, but these errors were encountered: