Bug 37 - Another SGI warning
Summary: Another SGI warning
Status: CLOSED FIXED
Alias: None
Product: Portable OpenSSH
Classification: Unclassified
Component: Miscellaneous (show other bugs)
Version: -current
Hardware: All IRIX
: P2 trivial
Assignee: Kevin Steves
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2001-12-11 15:37 AEDT by Tom Holroyd
Modified: 2004-04-14 12:24 AEST (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tom Holroyd 2001-12-11 15:37:11 AEDT
I might have missed this one before, sorry.
It's similar to the others and could be resolved
the same way, with an extra enum for "invalid":

cc-1185 cc: WARNING File = log.c, Line = 71
  An enumerated type is mixed with another type.

        { NULL, 0 }
                ^

cc-1185 cc: WARNING File = log.c, Line = 88
  An enumerated type is mixed with another type.

        { NULL, 0 }
                ^

No additional warnings have been introduced, only
a few remain that were previously reported and
explicitly not fixed.
Comment 1 Kevin Steves 2001-12-20 08:01:59 AEDT
i'm not sure we want to try to silence every warning for valid C.
in the other cases it made sense, i don't really know about this
one.  but we could add 2 _UNKNOWN enums to silence this.
Comment 2 Kevin Steves 2002-01-07 04:08:42 AEDT
does this resolve the warnings on SGI?

Index: log.c
===================================================================
RCS file: /cvs/src/usr.bin/ssh/log.c,v
retrieving revision 1.19
diff -u -r1.19 log.c
--- log.c	19 Dec 2001 07:18:56 -0000	1.19
+++ log.c	5 Jan 2002 22:50:08 -0000
@@ -65,7 +65,7 @@
 	{ "LOCAL5",	SYSLOG_FACILITY_LOCAL5 },
 	{ "LOCAL6",	SYSLOG_FACILITY_LOCAL6 },
 	{ "LOCAL7",	SYSLOG_FACILITY_LOCAL7 },
-	{ NULL, 0 }
+	{ NULL, (SyslogFacility)0 }
 };
 
 static struct {
@@ -82,7 +82,7 @@
 	{ "DEBUG1",	SYSLOG_LEVEL_DEBUG1 },
 	{ "DEBUG2",	SYSLOG_LEVEL_DEBUG2 },
 	{ "DEBUG3",	SYSLOG_LEVEL_DEBUG3 },
-	{ NULL, 0 }
+	{ NULL, (LogLevel)0 }
 };
 
 static void	 do_log(LogLevel level, const char *fmt, va_list args);
Comment 3 Kevin Steves 2002-01-17 15:36:15 AEDT
tom confirmed, commited fix.
Comment 4 Damien Miller 2004-04-14 12:24:17 AEST
Mass change of RESOLVED bugs to CLOSED