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
Fix #8221: Subtitle - No Bridge Type Given Size #8240
Conversation
The error box for going over max the lengths for each of the bridge type was given the "...bridge too long" subtitle.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a quick read, didn't check the actual change.
src/bridge_gui.cpp
Outdated
@@ -426,9 +426,12 @@ void ShowBuildBridgeWindow(TileIndex start, TileIndex end, TransportType transpo | |||
default: break; | |||
} | |||
|
|||
bool any_available = 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bool any_available = 0; | |
bool any_available = false; |
src/bridge_gui.cpp
Outdated
@@ -437,8 +440,14 @@ void ShowBuildBridgeWindow(TileIndex start, TileIndex end, TransportType transpo | |||
/* Add to terraforming & bulldozing costs the cost of the | |||
* bridge itself (not computed with DC_QUERY_COST) */ | |||
item.cost = ret.GetCost() + (((int64)tot_bridgedata_len * _price[PR_BUILD_BRIDGE] * item.spec->price) >> 8) + infra_cost; | |||
any_available = 1; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
any_available = 1; | |
any_available = true; |
Per glx22's suggestion.
The error box for going over max the lengths for each of the bridge types was given the "...bridge too long" subtitle.