Complete.Org: Mailing Lists: Archives: freeciv-dev: December 2001:
[Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149)
Home

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

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: Gregory Berkolaiko <gberkolaiko@xxxxxxxxxxx>
Cc: Petr Baudis <pasky@xxxxxxxxxxx>, freeciv-dev@xxxxxxxxxxx, bugs@xxxxxxxxxxxxxxxxxxx
Subject: [Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149)
From: Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>
Date: Fri, 21 Dec 2001 18:41:33 +0100
Reply-to: rf13@xxxxxxxxxxxxxxxxxxxxxx

On Fri, Dec 21, 2001 at 04:37:11PM +0000, Gregory Berkolaiko wrote:
>  --- Petr Baudis <pasky@xxxxxxxxxxx> wrote: 
> > Dear diary, on Fri, Dec 21, 2001 at 05:08:29PM CET, I got a letter,
> > where Gregory Berkolaiko <gberkolaiko@xxxxxxxxxxx> told me, that...
> > > 
> > > this is suspicious.
> > > maybe try and put assert(!happy) here?
> > We shouldn't. It's not a requirement. When (!happy), you just actually
> > drained whole effect of the building.
> 
> I meant put an assert, run a few games and see if it ever fails.
> of course it's not a requirement, just a testing assert.
> you can do it in gdb too of course
> 
> > > > > In general the patch is very good and I can only pity it's sad
> > fate.
> > > > Ehm? What? Sad fate?! ;)
> > > > 
> > > 
> > > I have a bad premonition about your patch...
> > I want to get it into official CVS and if it will be possible I hope I
> > will get
> > it there :). I just don't see anything controversial on it (hey, I even
> > try to
> > abide coding rules), I see sufficient (for me) feedback for it, so
> > what? 
> 
> we will see.
> I personally think that your patch has everything: it is useful, it got
> +ve reviews, it even abides coding rules.  

> But it doesn't seem to be sufficient here.

Come on. You know how slow I'm. But you should be able to see how I
advance in the patches. 

BTW: The fact nobody noticed that #1127 is broken is not a good sign.

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "Transported to a surreal landscape, a young girl kills the first woman
  she meets and then teams up with three complete strangers to kill again."
    -- TV listing for the Wizard of Oz in the Marin Independent Journal


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