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 #5405: Aircraft could route to depots outside their range #7104

Conversation

SamuXarick
Copy link
Contributor

This fix is to avoid out of range when using service at nearest hangar orders

@SamuXarick
Copy link
Contributor Author

SamuXarick commented Jan 25, 2019

For this PR, I need a bit of help, because I'm not 100% sure if it works. Didn't test what happens when conditional orders are involved, for example.

Reference: #6925 (comment)

Savegame: https://github.com/OpenTTD/OpenTTD/files/2444256/Windside.Heath.Transport.1985-03-05.zip

  • just follow the helicopter complete its schedulle, it gets destination too far after order 4.

@LordAro LordAro changed the title Fix #5405: Determine the correct last and next destinations and then ensure these are within range of each other Fix #5405: Aircraft routing to depots outside their range Jan 31, 2019
@LordAro LordAro changed the title Fix #5405: Aircraft routing to depots outside their range Fix #5405: Aircraft could route to depots outside their range Jan 31, 2019
@SamuXarick SamuXarick force-pushed the ensure-last-and-next-destinations-are-within-range branch from 15ca0bb to 2e7b716 Compare March 1, 2019 15:03
@stale
Copy link

stale bot commented Mar 31, 2019

This pull request has been automatically marked as stale because it has not had any activity in the last month.
Please feel free to give a status update now, ping for review, or re-open when it's ready.
It will be closed if no further activity occurs within 7 days.
Thank you for your contributions.

@stale stale bot added the stale Stale issues label Mar 31, 2019
@SamuXarick SamuXarick force-pushed the ensure-last-and-next-destinations-are-within-range branch from 2e7b716 to 874e279 Compare April 1, 2019 02:52
@stale stale bot removed the stale Stale issues label Apr 1, 2019
@SamuXarick SamuXarick force-pushed the ensure-last-and-next-destinations-are-within-range branch 2 times, most recently from ab87207 to 100a10d Compare April 13, 2019 21:38
LordAro
LordAro previously approved these changes Apr 27, 2019
Copy link
Member

@LordAro LordAro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd like someone else to check it over, but I have no issues with this in principle

@LordAro LordAro requested a review from michicc June 23, 2019 17:18
src/aircraft_cmd.cpp Outdated Show resolved Hide resolved
Determine the correct last and next destinations and then ensure these are within range of each other
This fix is to avoid out of range when using service at nearest hangar orders
@SamuXarick SamuXarick force-pushed the ensure-last-and-next-destinations-are-within-range branch from 100a10d to dfbe6ae Compare July 17, 2019 19:26
@LordAro LordAro requested a review from michicc August 17, 2019 20:53
@LordAro LordAro merged commit ccb4c37 into OpenTTD:master Nov 2, 2019
@SamuXarick SamuXarick deleted the ensure-last-and-next-destinations-are-within-range branch January 2, 2020 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants