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

Jump to bottom

Fix f66baa44: for-loop is no longer increasing "i" #8436

Merged
merged 1 commit into from Dec 26, 2020
Merged

Fix f66baa44: for-loop is no longer increasing "i" #8436

merged 1 commit into from Dec 26, 2020

Conversation

TrueBrain
Copy link
Member

Motivation / Problem

When you go to the AI list, the first AI was always selected instead of the current selected on.

Description

During conversion it was overlooked that the for-loop used to do
this. Oops.

See f66baa4#diff-65e506358d200c3adbd2679e9f04fdb017459a79f08852173dd9fc5964859dbeL90 for the original code. See how sneaky the i++ was hiding there? Bah :P

@TrueBrain
Fix f66baa4: for-loop is no longer increasing "i"
abcc2bd 
During conversion it was overlooked that the for-loop used to do
this. Oops.
@LordAro LordAro merged commit 8fa2a67 into OpenTTD:master Dec 26, 2020
@TrueBrain TrueBrain deleted the fix_missing_i_increase branch December 26, 2020 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LordAro LordAro LordAro approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@TrueBrain @LordAro