[Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149)

[Raahul Kumar <raahul_da_man@xxxxxxxxx>]

--- Petr Baudis <pasky@xxxxxxxxxxx> wrote:
> Dear diary, on Fri, Dec 21, 2001 at 12:41:32PM CET, I got a letter,
> where Raahul Kumar <raahul_da_man@xxxxxxxxx> told me, that...
> > > > > +  /* If units are making us unhappy, count that too. */
> > > > > +  sad += pcity->ppl_unhappy[3] - pcity->ppl_unhappy[2];
> > > > > +  /* Angry citizens has to be counted double!? */
> > > > 
> > > > they _have_ to be counted double
> > > Hey, this was in original source too! ;))
> >
> > I think that is a bad comment. Maybe something like One angry citizens is
> > equal to two unhappy citizen, because an angry citizen has to be made
> > unhappy, then he can be made content. The man is a ticking time bomb.
> That makes sense. Updated patch attached :).
> 

Good. I approve fully of the patch now. Unless someone else has noticed any
bugs, I vote for inclusion.

__________________________________________________
Do You Yahoo!?
Send your FREE holiday greetings online!
http://greetings.yahoo.com