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

Farm fields 'die' if their farm shuts down #82

Closed
DorpsGek opened this issue Mar 25, 2006 · 3 comments
Closed

Farm fields 'die' if their farm shuts down #82

DorpsGek opened this issue Mar 25, 2006 · 3 comments
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

Comments

@DorpsGek
Copy link
Member

peter1138 opened the ticket and wrote:

This patch causes farm fields to be removed after their farm has shut down.

This is achieved by storing the industry id of the farm in m2 for the fields. If the farm is removed, those fields have m2 set to 0xffff (effectively INVALID_INDUSTRY, although that doesn't exist yet) and they are then removed during the tile loop.

No attempt has been made to convert existing fields, so only fields of new farms (or new games) with this patch will be removed.

Attachments

Reported version: trunk
Operating system: All


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

TrueBrain wrote:

Very nice patch. I updated it a bit, added INVALID_INDUSTRY and some comments.

Also, on load, all farmland is destroyed and new farmland is made for older savegames.

Very nice job :)

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/82#comment435

@DorpsGek
Copy link
Member Author

TrueBrain wrote:

And yet an other new version, which will be applied after past2090 commit later tonight.

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/82#comment436

@DorpsGek
Copy link
Member Author

Bjarni closed the ticket.

Reason for closing: Implemented


This comment was imported from FlySpray: https://bugs.openttd.org/task/82

@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 6, 2018
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
Projects
None yet
Development

No branches or pull requests

1 participant