Complete.Org: Mailing Lists: Archives: freeciv-dev: September 2001:
[Freeciv-Dev] Re: Settlers turns left calculation in GUI is wrong. (PR#9
Home

[Freeciv-Dev] Re: Settlers turns left calculation in GUI is wrong. (PR#9

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: freeciv-dev@xxxxxxxxxxx
Cc: bugs@xxxxxxxxxxxxxxxxxxx
Subject: [Freeciv-Dev] Re: Settlers turns left calculation in GUI is wrong. (PR#981)
From: rf13@xxxxxxxxxxxxxxxxxxxxxx
Date: Thu, 27 Sep 2001 03:41:52 -0700 (PDT)

On Thu, Sep 27, 2001 at 12:05:39PM +0200, Stepan Roh wrote:
> 
> 
> On Thu, 27 Sep 2001, Raimar Falke wrote:
> 
> > >    struct tile *ptile = map_get_tile(x, y);
> > >
> > >    memset(activity_total, 0, sizeof(activity_total));
> > >    memset(activity_units, 0, sizeof(activity_units));
> > >
> > >    unit_list_iterate(ptile->units, punit) {
> > > -    activity_total[punit->activity] += punit->activity_count;
> > >      mr = get_unit_type(punit->type)->move_rate;
> > > -    activity_units[punit->activity] += mr ? mr / SINGLE_MOVE : 1;
> >
> > Mhh what kinds of values does the term "mr ? mr / SINGLE_MOVE : 1" can
> > get? Can mr really be 0? Yes this isn't in your patch and I should
> > have catched this in the old patch.
> 
> It comes from http://www.freeciv.org/lxr/source/server/unittools.c#L944.

So what about the attached patch? It even works ;)

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
  What's nice about GUI is that you see what you manipulate.
  What's bad about GUI is that you can only manipulate what you see.

Attachment: move_rate1.diff
Description: Text document


[Prev in Thread] Current Thread [Next in Thread]