[Freeciv-Dev] Re: [PATCH] Re: Short stupid question (PR#569)

[Gaute B Strokkenes <gs234@xxxxxxxxx>]

Erik Sigra <freeciv@xxxxxxx> writes:

> Gaute B Strokkenes wrote:
> > 
> > Erik Sigra <freeciv@xxxxxxx> writes:
> > 
> > > Erik Sigra wrote:
> > >
> > > Now I tested it and got a quite funny result. See
> > > <ftp://ftp.freeciv.org/freeciv/incoming/hmap_fix.png>. Especially
> > > note the mini-map. (Not that it would be uniteresting to play on
> > > such a map, but I suggest you take a look at the code again.)
> > 
> > Quick question: you do realise that hmap() changed from the address of
> > a particular array entry, to the actual entry?
> 
> Sure. Have you found the problem that causes the effect in the
> screenshot above?

Ya.  The problem was a couple of accidental x <-> y transposes.
Updated patch attached.

hmap_fix-2.diff
Description: Text Data

-- 
Big Gaute (not to be confused with LG)
This is PLEASANT!