Complete.Org: Mailing Lists: Archives: freeciv-data: March 2003:
[freeciv-data] Re: [Freeciv-Dev] (PR#3898) Clarify include order
Home

[freeciv-data] Re: [Freeciv-Dev] (PR#3898) Clarify include order

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
Cc: freeciv-data@xxxxxxxxxxx
Subject: [freeciv-data] Re: [Freeciv-Dev] (PR#3898) Clarify include order
From: "Raimar Falke" <rf13@xxxxxxxxxxxxxxxxx>
Date: Sun, 30 Mar 2003 23:52:37 -0800
Reply-to: freeciv-data@xxxxxxxxxxx

On Sat, Mar 29, 2003 at 04:06:33PM -0800, Mike Kaufman wrote:
> On Sat, Mar 29, 2003 at 03:34:55PM -0800, Raimar Falke wrote:
> > On Sat, Mar 29, 2003 at 01:29:30PM -0800, Mike Kaufman wrote:
> > > > +
> > > > +    1) config.h (see below)
> > > > +    2) system include files which are OS-independent (part of
> > > > +       C-standard or POSIX)
> > > > +    3) system include files which are OS-dependent or conditional
> > > > +       includes
> > > > +    4) freeciv include files from common/
> > > > +    5) freeciv include files from common/aicore/
> > > 
> > > 4 and 5 possibly should be grouped.
> > > 
> > > > +    6) freeciv include files from client/
> > > > +    7) freeciv include files from client/agents/
> > > > +    8) freeciv include files from client/gui-*/
> > > > +    9) freeciv include files from client/include/
> > > 
> > > 8 and 9 should be grouped. 
> > > 
> > > > +   10) freeciv include files from server/
> > > > +   11) freeciv include files from ai/
> > > > +
> 
> how about 
> 
> 4) include files from common/ and common/aicore
> 5) include files from client/ client/include and client/agents
> 6) include files from client/gui-*
> 7) include files from server/
> 8) include files from ai/
> 
> now you don't really have to remember which comes first (client/agents or
> client/include etc)

Ok. Will you apply it?

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
  Microsoft does have a year 2000 problem. I'm part of it. I'm running Linux.




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