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
Patch to guard for NO_DEBUG_MESSAGES #6652
Comments
frosch wrote:
This comment was imported from FlySpray: https://bugs.openttd.org/task/6652#comment14866 |
Borg wrote:
Attachments
This comment was imported from FlySpray: https://bugs.openttd.org/task/6652#comment14874 |
The genworld.cpp seems valid :D |
Okay, this NO_DEBUG_MESSAGES is a bit weird. First of all, you cannot change it without modifying files. But also, debug_desync_level does a lot more than just spitting out messages. But the other debug_NNN_level mostly only changes what is being printed. A very weird combination, that is hard to untangle. Not sure that the supplied patch is the best way. Possibly it is easier to just remove NO_DEBUG_MESSAGES. |
Given any speed issue cannot be attributed to checking for _debug_NNN_level, removing this is a safe action This fixes OpenTTD#6652.
Given any speed issue cannot be attributed to checking for _debug_NNN_level, removing this is a safe action This fixes OpenTTD#6652.
Borg opened the ticket and wrote:
Attachments
Reported version: 1.7.2
Operating system: All
This issue was imported from FlySpray: https://bugs.openttd.org/task/6652
The text was updated successfully, but these errors were encountered: