[3224] in Kerberos

home help back first fref pref prev next nref lref last post

system include files

daemon@ATHENA.MIT.EDU (dewey@MIT.EDU)
Mon May 2 21:39:04 1994

From: dewey@MIT.EDU
Date: Mon, 2 May 94 21:23:00 -0400
To: crosis@MIT.EDU
Cc: rbfinton@MIT.EDU, nexus6@MIT.EDU, adasilva@MIT.EDU, edreall@MIT.EDU,
        kerberos@MIT.EDU
In-Reply-To: crosis@MIT.EDU's message of Mon, 2 May 94 20:37:42 -0400 <9405030037.AA23746@pesto.MIT.EDU>


Yep, it's those idiots at DEC.  gcc warns you if you have a /* inside
a comment, because some people try to do nested comments.  Comments do
not next.  Example:

/* printf("this is a test\n"); /* a test printf */   */

This is an example of "commenting out" a line incorrectly, the lines
referenced in your error message were:
/* #define SIOCSCREENON	_IOWR('i', 49, int)	/* screend, net/gw_screen.h */
/* #define SIOCSCREEN	_IOWR('i', 50, struct screen_data)	/* screend */
/* #define SIOCSCREENSTATS _IOR('i', 51, struct screen_stats)	/* screend */

As you can see, they didn't try to nest the comments, but they do have
/* in the comments.  A better way of doing things is to comment out
code like this:

#if 0
printf("This is a test\n");
#endif

This avoids the nested comment stuff.

Note: These are warnings only.  They are not errors.  Your gcc line is
treating them as errors because you are using -Werror, which means to
treat warnings as errors.  That is a great idea in general, and I
think that the people at DEC should have made their code squeaky clean
so that it would work, but they didn't.  Remove the -Werror from your
comple line.

Dewey

home help back first fref pref prev next nref lref last post