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: Replace duplicate town name "Huacho" #7038

Merged
merged 1 commit into from Jan 12, 2019

Conversation

nikolas
Copy link
Member

@nikolas nikolas commented Jan 12, 2019

Huacho appears twice in the Spanish town names list. This change removes
the second one.

This will have the effect of altering many town names in any older
savegame using Latin American town names. I think that's an
acceptable side effect for a new OpenTTD release, but if you have a
better solution let me know. Maybe some type of null "Dummy" value here?

@James103
Copy link
Contributor

Why don't you pick a random Spanish town name that's not already on the Spanish town name list to replace the second "Huacho"?

Huacho appears twice in the Spanish town names list. This change removes
the second one, and replaces it with a new one: Medellin.
@nikolas
Copy link
Member Author

nikolas commented Jan 12, 2019

Good idea! That's a better solution - I've replaced it with Medellín. That will have a lot less effect on existing savegames.

@nikolas nikolas changed the title Fix: Remove duplicate town name "Huacho" Fix: Replace duplicate town name "Huacho" Jan 12, 2019
Copy link
Contributor

@andythenorth andythenorth left a comment

Choose a reason for hiding this comment

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

Looks good. For diligence, I also checked the unicode escape, it's correctly "í".

@planetmaker planetmaker merged commit 82e530b into OpenTTD:master Jan 12, 2019
nielsmh pushed a commit to nielsmh/OpenTTD that referenced this pull request Mar 11, 2019
Huacho appears twice in the Spanish town names list. This change removes
the second one, and replaces it with a new one: Medellin.
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

4 participants