Patch: American Town Names

Forum for technical discussions regarding development. If you have a general suggestion, problem or comment, please use one of the other forums.

Moderator: OpenTTD Developers

Post Reply
Johnny B Goode
Director
Director
Posts: 534
Joined: 17 Mar 2007 16:36

Patch: American Town Names

Post by Johnny B Goode »

I felt the British town names sounded kind of foreign to me, so I wrote this patch. Keep in mind, I was operating straight out of the Dev's Black Book, and this is my first patch. Also, I haven't tested it becasue my patch program is acting weirdly.
Attachments
american_town_names.patch
(1.46 KiB) Downloaded 137 times
Last edited by Johnny B Goode on 05 May 2007 18:05, edited 3 times in total.
Jezulkim wrote:Credits:
- Johnny B Goode, for digging this thread up and giving me an idea for this update.
- Mr. Owen Rudge, for his TT website where I grabbed the names.
Yeah! I've been credited! (Thanks, Jezulkim)
Winner of Most Annoying Member Award 2007 - Runner-up for Biggest Spammer 2007
My blog - Diary Of A Madman
Image
Image
User avatar
Maedhros
OpenTTD Developer
OpenTTD Developer
Posts: 603
Joined: 30 Mar 2006 18:24
Location: Durham, UK

Re: Patch: American Town Names

Post by Maedhros »

Johnny B Goode wrote:Also, I haven't tested it becasue my patch program is acting weirdly.
You're not kidding - that's not a particularly useful (or even valid) patch, I'm afraid. :shock:

Unfortunately it wouldn't work if it was a valid patch since the arrays of names you've defined are different to the ones you're referencing in MakeAmericanTownName. Also I have a feeling that the code wouldn't do what you were expecting anyway as it would pick a name from either of the arrays, not both of them.
No-one's more important than the earthworm.
Johnny B Goode
Director
Director
Posts: 534
Joined: 17 Mar 2007 16:36

Post by Johnny B Goode »

I fixed the invalid array calls and the generator function. Anything else needed?

EDIT: I also fixed some other errors as well.
Attachments
american_town_names.patch
(1.46 KiB) Downloaded 151 times
Jezulkim wrote:Credits:
- Johnny B Goode, for digging this thread up and giving me an idea for this update.
- Mr. Owen Rudge, for his TT website where I grabbed the names.
Yeah! I've been credited! (Thanks, Jezulkim)
Winner of Most Annoying Member Award 2007 - Runner-up for Biggest Spammer 2007
My blog - Diary Of A Madman
Image
Image
DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

That *still* isn't a valid patch file.
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
User avatar
Zuu
OpenTTD Developer
OpenTTD Developer
Posts: 4553
Joined: 09 Jun 2003 18:21
Location: /home/sweden

Post by Zuu »

Shouldn't there be a space between name1 and name2? Or are American town names written w/o space?


Also you need to make the patch using svn diff to get somewhere with your patch I'm afraid. Your patch don't specify where your new code should be added in the files. Basically edit the files straight ahead and then make a svn diff.
My OpenTTD contributions (AIs, Game Scripts, patches, OpenTTD Auto Updater, and some sprites)
Junctioneer (a traffic intersection simulator)
Johnny B Goode
Director
Director
Posts: 534
Joined: 17 Mar 2007 16:36

Post by Johnny B Goode »

Eh, screw this. Hopefully I can ask someone who can code for s*** to fix it.
Jezulkim wrote:Credits:
- Johnny B Goode, for digging this thread up and giving me an idea for this update.
- Mr. Owen Rudge, for his TT website where I grabbed the names.
Yeah! I've been credited! (Thanks, Jezulkim)
Winner of Most Annoying Member Award 2007 - Runner-up for Biggest Spammer 2007
My blog - Diary Of A Madman
Image
Image
Post Reply

Return to “OpenTTD Development”

Who is online

Users browsing this forum: No registered users and 5 guests