[Freeciv-Dev] Re: (PR#9869) Bug: DDEBUG changes savegames

["Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>]

<URL: http://rt.freeciv.org/Ticket/Display.html?id=9869 >

Gregory Berkolaiko wrote:

> I am not sure which is the better way to fix this: do temporary
> assignments inside index_to* functions or do it in the caller.  The first
> way might give more work to compiler (optimizing), the second will give
> more work to us (reviewing all uses).

We can't do temporary assignments unless we make it an inline function. 
  Changing the users is fine but we also have to rename it as 
INDEX_TO_MAP_POS.  I would prefer the former but either is fine.

jason