Re: [PATCH 1/2] server/red_worker.h: order includes correcty

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Sep 01, 2013 at 03:24:32AM -0400, Alon Levy wrote:
> > Do we document anywhere what is a 'correct' order? I assume it's config.h,
> > alphabetically ordered system headers (with subdirs coming last), and
> > finally local headers?
> > I assume this is pretty standard practice, but I always have to think to
> > remember exactly how to order things :)
> 
> There is a coding standard, in pdf form, it should have this reference.

Ah right, I had totally forgotten about that doc, I was looking for it
along with spice sources ;) It does not seem to talk about header orders
though.

> The 'correct' order, as I (and Hans too it seems) see it is:
> system includes
> external libraries includes
> internal not same file name includes
> file (i.e. s/filename.c/filename.h/) include, if any

Works for me (any order would I guess ;) I was just wondering where this
'correct' order was defined for future reference.

Christophe

Attachment: pgp3gfHwkGD03.pgp
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]