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

Airports: when placing an airport, consider airport tile table tiles only. #6613

Closed
DorpsGek opened this issue Aug 19, 2017 · 1 comment
Closed
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay stale Stale issues

Comments

@DorpsGek
Copy link
Member

adf88 opened the ticket and wrote:

When placing an airport, check for flat land and clear tiles that belong to the airport tile table only rather then iterating whole airport rectangle.

There is no point in checking other tiles (other then these in the tile table) - they can be altered after the airport is built anyway.

This patch allows e.g. to always re-create an airport on the same location, also airports with non-uniform (non-rectangular) layouts, even thought some obstacles appeared on unoccupied tiles inside airport rectangle.

Attachments

Reported version: trunk
Operating system: All


This issue was imported from FlySpray: https://bugs.openttd.org/task/6613
@DorpsGek
Copy link
Member Author

DorpsGek commented Sep 3, 2017

andythenorth wrote:

Retitled only because I'm making lists of related items based on ticket title prefixes ;)


This comment was imported from FlySpray: https://bugs.openttd.org/task/6613#comment14746

@DorpsGek DorpsGek added Core flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay labels Apr 7, 2018
@frosch123 frosch123 removed the Core label Apr 14, 2018
@andythenorth andythenorth added the stale Stale issues label Jan 5, 2019
Eddi-z added a commit to Eddi-z/OpenTTD that referenced this issue Jan 7, 2019
LordAro pushed a commit that referenced this issue Jan 11, 2019
nielsmh pushed a commit to nielsmh/OpenTTD that referenced this issue Mar 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay stale Stale issues
Projects
None yet
Development

No branches or pull requests

3 participants