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

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