http://bugs.freeciv.org/Ticket/Display.html?id=39385 >
I don't understand your changes to the first functions, but I have
some comments on the other changes.
I did this with the field width wrong as well, but I think we should
reserve a width up to 5 for build costs. I can imagine wonders costin
http://bugs.freeciv.org/Ticket/Display.html?id=39385 >
On 6/7/2007 8:11 AM, Ulrik Sverdrup wrote:
> http://bugs.freeciv.org/Ticket/Display.html?id=39385 >
>
> Here is a preliminary patch. I don't know the code around this, but it
> seems to be well designed, so this should be all there is to it.
James Supancic wrote:
> In resource_name_translation, in terrain.c you are altering the value
> of a const parameter.
> This is not allowed, and gcc 4.1 tells us this with:
> terrain.c: In function 'resource_name_translation':
> terrain.c:244: error: assignment of read-only location
>
> I'm not su
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
http://bugs.freeciv.org/Ticket/Display.html?id=39385 >
Ulrik Sverdrup wrote:
> I did this with the field width wrong as well, but I think we should
> reserve a width up to 5 for build costs. I can imagine wonders costing
> more than 1 if you buy them from scratch in the derfault ruleset.
> Wi