Skip to content
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 #8055: Fix crash when the road toolbar is open, but the availabil… #8058

Merged
merged 1 commit into from Apr 5, 2020

Conversation

@stormcone
Copy link
Contributor

@stormcone stormcone commented Apr 3, 2020

…ity of the current road type has changed.

This patch is only fixes the crash part of the issue.

src/road_gui.cpp Outdated Show resolved Hide resolved
src/road_gui.cpp Outdated Show resolved Hide resolved
src/road_gui.cpp Outdated Show resolved Hide resolved
glx22
glx22 previously approved these changes Apr 5, 2020
Copy link
Contributor

@glx22 glx22 left a comment

LGTM, commit message may need to be rewritten to include reference to c02ef3e and shortening

…ut the availability of the current road type has changed.
@stormcone
Copy link
Contributor Author

@stormcone stormcone commented Apr 5, 2020

Do you have any suggestions, or could you rewrite it when you merge?

glx22
glx22 approved these changes Apr 5, 2020
Copy link
Contributor

@glx22 glx22 left a comment

Yes it can be rewritten on merge

@LordAro LordAro merged commit 2514f43 into OpenTTD:master Apr 5, 2020
8 checks passed
LordAro added a commit to LordAro/OpenTTD that referenced this issue Apr 13, 2020
LordAro added a commit that referenced this issue Apr 13, 2020
douiwby added a commit to douiwby/OpenTTD that referenced this issue Apr 16, 2020
@stormcone stormcone deleted the pr-8055 branch Aug 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants