Complete.Org: Mailing Lists: Archives: freeciv-dev: February 2004:
[Freeciv-Dev] Re: (PR#7405) list cleanup
Home

[Freeciv-Dev] Re: (PR#7405) list cleanup

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: per@xxxxxxxxxxx
Subject: [Freeciv-Dev] Re: (PR#7405) list cleanup
From: "Raimar Falke" <i-freeciv-lists@xxxxxxxxxxxxx>
Date: Wed, 11 Feb 2004 12:19:29 -0800
Reply-to: rt@xxxxxxxxxxx

<URL: http://rt.freeciv.org/Ticket/Display.html?id=7405 >

On Tue, Feb 10, 2004 at 01:02:37PM -0800, Per I. Mathisen wrote:
> 
> <URL: http://rt.freeciv.org/Ticket/Display.html?id=7405 >
> 
> On Tue, 10 Feb 2004, Raimar Falke wrote:
> > > Also, if an unlink_all() becomes necessary to actually remove all list
> > > data, as my genlist patch does, calling unlink_all() on an empty list is
> > > not superfluous but necessary.
> >
> > Uhh. Bad idea. It changes the semantics of the function thus code like:
> >
> > unlink_all()
> > insert()
> >
> > becomes broken.
> 
> That is never used anywhere, AFAICT.

Well you can prove this? Or catch it? Looks like we exchange one
potential problem (code is currently in a working state) with another
one.

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "I haven't lost my mind - it's backed up on tape somewhere."




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