Complete.Org: Mailing Lists: Archives: freeciv-dev: November 2002:
[Freeciv-Dev] Re: (PR#2350) Remove dependencies
Home

[Freeciv-Dev] Re: (PR#2350) Remove dependencies

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: rf13@xxxxxxxxxxxxxxxxx
Subject: [Freeciv-Dev] Re: (PR#2350) Remove dependencies
From: "Jason Short via RT" <rt@xxxxxxxxxxxxxx>
Date: Fri, 15 Nov 2002 17:26:14 -0800
Reply-to: rt@xxxxxxxxxxxxxx

Mike Kaufman via RT wrote:
> I don't like this change. It adds files to common/, it adds an extra layer
> of indirection to header files, simply to improve compilation time if a
> certain file is changed?? Hmm.

The addition of a file or two seems OK, but this sets a bad precedent: 
one file per iterator.  Since there are more iterators that should be 
implemented, does this mean each of them should get its own files?

Such a change would be fine with me if it moved all iterators into a 
single file.

jason




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