Re: [Freeciv-Dev] (PR#39383) BUG: terrain and resource name translation

2007-06-08 Thread William Allen Simpson
http://bugs.freeciv.org/Ticket/Display.html?id=39383 > William Allen Simpson wrote: > There's a warning, because the current resource list is const, and this > modifies the pointer. Updating the const can happen later. > trunk revision 12977 S2_1 revision 12978 Remaining "const struct resource

Re: [Freeciv-Dev] (PR#39383) BUG: terrain and resource name translation

2007-06-05 Thread William Allen Simpson
http://bugs.freeciv.org/Ticket/Display.html?id=39383 > trunk revision 12972 S2_1 revision 12973 As S2_1 required a number of trivial changes, here's the final patch for the record (after 12972, making 12973). In particular, where translated strings already exist, they are painstakingly maintain

Re: [Freeciv-Dev] (PR#39383) BUG: terrain and resource name translation

2007-06-03 Thread William Allen Simpson
http://bugs.freeciv.org/Ticket/Display.html?id=39383 > In this version, I moved the translation into the accessor functions, as this seemed to be better than spread around in 3 places, especially as the other places in the code forgot to test for NULL and '\0' (empty string). Comments in the code

[Freeciv-Dev] (PR#39383) BUG: terrain and resource name translation

2007-06-03 Thread William Allen Simpson
http://bugs.freeciv.org/Ticket/Display.html?id=39383 > While trying to debug some of my terrain changes, it was frustrating that some error messages had empty (missing/blank) fields. Numerous messages tried to insert a translated name into log messages that are not translated. (I'm running witho