[Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149)
[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
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
- [Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149), (continued)
- [Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149), Raahul Kumar, 2001/12/21
- [Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149), Petr Baudis, 2001/12/21
- [Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149), Raahul Kumar, 2001/12/22
- [Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149), Gregory Berkolaiko, 2001/12/21
- [Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149), Petr Baudis, 2001/12/21
- [Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149), Gregory Berkolaiko, 2001/12/21
- [Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149),
Raimar Falke <=
- [Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149), Gregory Berkolaiko, 2001/12/21
- [Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149), Jason Short, 2001/12/21
- [Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149), Petr Baudis, 2001/12/21
- [Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149), Raimar Falke, 2001/12/22
- [Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149), Petr Baudis, 2001/12/22
- [Freeciv-Dev] Another Maintainer, Raahul Kumar, 2001/12/22
- [Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149), Ross W. Wetmore, 2001/12/21
- [Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149), Raimar Falke, 2001/12/22
- [Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149), Petr Baudis, 2001/12/22
- [Freeciv-Dev] Re: [PATCH] advdomestic.c cleanup (PR#1149), Ross W. Wetmore, 2001/12/22
|
|