[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r33945 - trunk/subversion/libsvn_subr

From: Jens Seidel <jensseidel_at_users.sf.net>
Date: Wed, 29 Oct 2008 14:40:27 +0100

On Wed, Oct 29, 2008 at 04:46:59AM -0700, Greg Stein wrote:
> We normally group headers like:
>
> #include <stdlib.h> /* system headers */
> #include <another.h>
>
> #include <apr_pools.h> /* other package headers */
> #include <apr_file_io.h>
>
> #include "svn_types.h" /* our public headers */
> #include "svn_error.h"
>
> #include "opt.h" /* our private headers */
> #include "checksum.h"

You all probably know it but let me add it nevertheless:

It is preferred to put system and other package headers last because
it's easier to spot missing includes in own files this way ...

Jens

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-10-29 14:41:15 CET

This is an archived mail posted to the Subversion Dev mailing list.