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

coastal cities/towns #2635

Closed
DorpsGek opened this issue Feb 15, 2009 · 5 comments
Closed

coastal cities/towns #2635

DorpsGek opened this issue Feb 15, 2009 · 5 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

db48x opened the ticket and wrote:

This patch increases the prevalence of coastal towns by moving randomly generated towns that land in the water up onto the nearest land rather than simply discarding them. The patch only affects world generation. See also http://www.tt-forums.net/viewtopic.php?f=33&t=41856.

Reported version: trunk
Operating system: All

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

db48x wrote:

maybe this time it will upload?

Attachments

This comment was imported from FlySpray: https://bugs.openttd.org/task/2635#comment5585

@DorpsGek
Copy link
Member Author

db48x wrote:

now with improvements suggested by the fellows on irc, such as spaces->tabs, braces in the right places, and reusing GetNearestWaterDistance instead of writing my own.

Attachments

This comment was imported from FlySpray: https://bugs.openttd.org/task/2635#comment5586

@DorpsGek
Copy link
Member Author

db48x wrote:

Many style improvements, thanks to Yexo.

Attachments

This comment was imported from FlySpray: https://bugs.openttd.org/task/2635#comment5587

@DorpsGek
Copy link
Member Author

db48x wrote:

Rubidium pointed out an important omission. This patch ensures that all town locations are on the same grid if the user has selected the 2×2 or 3×3 grid as otherwise the road networks of neighboring towns will not mesh properly once they grow large enough to interpenetrate.

Attachments

This comment was imported from FlySpray: https://bugs.openttd.org/task/2635#comment5594

@DorpsGek
Copy link
Member Author

Rubidium closed the ticket.

Reason for closing: Implemented

In r16147

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

@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
Assignees
No one assigned
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
Milestone
No milestone
Development

No branches or pull requests
1 participant
@DorpsGek