[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: lib/xtime.h is included twice
From: |
Jim.Hyslop |
Subject: |
RE: lib/xtime.h is included twice |
Date: |
Thu, 3 Mar 2005 09:16:34 -0500 |
georg.schwarz@freenet.de wrote:
> Dear cvs developers,
>
> for cvs 1.11.19 and earlier, lib/xtime.h is included twice,
> which breaks
> compiling on IRIX 5.3.
> The following patch prevents this:
>
> --- lib/xtime.h.orig 2005-03-01 15:53:11.000000000 +0100
> +++ lib/xtime.h 2005-03-01 15:54:20.000000000 +0100
> @@ -12,6 +12,9 @@
> * functions
> */
>
> +#ifndef _XTIME_H_
> +#define _XTIME_H_
ISO C Standard reserves names beginning with underscore and an uppercase
character, or beginning with two underscores. I recommend changing this to
something like:
#ifndef XTIME_HEADER_INCLUDEd
#define XTIME_HEADER_INCLUDEd
(note: the lower-case d at the end is deliberate - it's my technique to
reduce the chances of a name collision.)
--
Jim Hyslop
Senior Software Designer
Leitch Technology International Inc. ( http://www.leitch.com )
Columnist, C/C++ Users Journal ( http://www.cuj.com/experts )