Complete.Org: Mailing Lists: Archives: freeciv-dev: April 2002:
[Freeciv-Dev] Re: a few notify fixes
Home

[Freeciv-Dev] Re: a few notify fixes

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: "Per I. Mathisen" <Per.Inge.Mathisen@xxxxxxxxxxx>, freeciv-dev@xxxxxxxxxxx
Subject: [Freeciv-Dev] Re: a few notify fixes
From: Raahul Kumar <raahul_da_man@xxxxxxxxx>
Date: Wed, 3 Apr 2002 03:36:53 -0800 (PST)

--- "Per I. Mathisen" <Per.Inge.Mathisen@xxxxxxxxxxx> wrote:
> On Wed, 3 Apr 2002, Raahul Kumar wrote:
> > Hmm? I don't know why you got rid of cancel pact and replaced it with
> treaty
> > broken.
> 
> For consistency with the other E_TREATY_*.

Consistency. I had heard of it. Until now, I've never expected to see it in the
wild.

> > > -  FIXME: this should be in plrhand
> 
> > You got rid of the FIXME. This function is fine where it is?
> 
> It has already been moved to plrhand.c. The comment is junk.

There are a lot of obsolete comments like that stuck in the code. Can we count
on Super Per to weed them out?


> Yours,
> Per
> 
> "While we're all in favor of Arafat doing more to suppress violence in
> the Middle East, we fear that he doesn't have all that much influence on
> the Israeli government. " --Sam Smith
> 
> 


__________________________________________________
Do You Yahoo!?
Yahoo! Tax Center - online filing with TurboTax
http://taxes.yahoo.com/


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