Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 9 Nov 2012 19:35:46 +0000 (UTC)
From:      Dimitry Andric <dim@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-9@freebsd.org
Subject:   svn commit: r242838 - stable/9/sys/contrib/ngatm/netnatm/msg
Message-ID:  <201211091935.qA9JZkXF041781@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: dim
Date: Fri Nov  9 19:35:46 2012
New Revision: 242838
URL: http://svnweb.freebsd.org/changeset/base/242838

Log:
  MFC r242623:
  
  In sys/contrib/ngatm/netnatm/msg/uni_ie.c, fix a few warnings from newer
  versions of clang 3.2, about comparing enum uni_cause values against
  integer constants which fall outside the enum range.  No functional
  change.

Modified:
  stable/9/sys/contrib/ngatm/netnatm/msg/uni_ie.c
Directory Properties:
  stable/9/sys/   (props changed)

Modified: stable/9/sys/contrib/ngatm/netnatm/msg/uni_ie.c
==============================================================================
--- stable/9/sys/contrib/ngatm/netnatm/msg/uni_ie.c	Fri Nov  9 19:32:37 2012	(r242837)
+++ stable/9/sys/contrib/ngatm/netnatm/msg/uni_ie.c	Fri Nov  9 19:35:46 2012	(r242838)
@@ -869,7 +869,7 @@ UNI_DECLARE_CAUSE_VALUES
 enum uni_diag
 uni_diag(enum uni_cause cause, enum uni_coding code)
 {
-	if (cause >= 128)
+	if ((int)cause >= 128)
 		return (UNI_DIAG_NONE);
 
 	if (code == UNI_CODING_NET)
@@ -925,9 +925,9 @@ print_cause(struct unicx *cx, struct uni
 	if (uni_print_iehdr("cause", &ie->h, cx))
 		return;
 
-	if (ie->cause < 128 && tab1[ie->cause].str)
+	if ((int)ie->cause < 128 && tab1[ie->cause].str)
 		strcpy(buf, tab1[ie->cause].str);
-	else if (ie->cause < 128 && tab2 != NULL && tab2[ie->cause].str != NULL)
+	else if ((int)ie->cause < 128 && tab2 != NULL && tab2[ie->cause].str != NULL)
 		strcpy(buf, tab2[ie->cause].str);
 	else {
 		sprintf(buf, "UNKNOWN-%u", ie->cause);
@@ -1059,7 +1059,7 @@ check_cause(struct uni_ie_cause *ie, str
 
 	const struct causetab *ptr;
 
-	if (ie->cause >= 128)
+	if ((int)ie->cause >= 128)
 		return (-1);
 
 	switch (ie->loc) {



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201211091935.qA9JZkXF041781>