This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

more signedness compiler warnings in cplus-dem.c


The enclosed patch avoids compiler warnings the way that existing code
in cplus-dem.c does.

A cleaner fix would be a fairly massive int->size_t conversion (or
unsigned int, perhaps).  I'm willing to embark on that but I thought I
should ask first whether people like that solution.

2000-02-07  Jim Kingdon  <kingdon@redhat.com>

	* cplus-dem.c: Put in more ugly casts to avoid compiler warnings.

Index: cplus-dem.c
===================================================================
RCS file: /cvs/src/src/libiberty/cplus-dem.c,v
retrieving revision 1.2
diff -u -r1.2 cplus-dem.c
--- cplus-dem.c	1999/05/11 12:42:12	1.2
+++ cplus-dem.c	2000/02/07 19:09:08
@@ -2439,7 +2439,7 @@
 	  break;
 	default:
 	  n = consume_count (mangled);
-	  if (n < 0 || n > strlen (*mangled))
+	  if (n < 0 || n > (int) strlen (*mangled))
 	    {
 	      success = 0;
 	      break;
@@ -2606,7 +2606,7 @@
 	{
 	  n = consume_count (mangled);
           if (n == -1
-	      || n > strlen (*mangled))
+	      || n > (int) strlen (*mangled))
 	    return 0;
 	  string_prependn (declp, *mangled, n);
 	  (*mangled) += n;
@@ -3365,7 +3365,7 @@
 	  int i;
 	  ++(*mangled);
 	  for (i = 0;
-	       (i < sizeof (buf) - 1 && **mangled && **mangled != '_');
+	       (i < (int) sizeof (buf) - 1 && **mangled && **mangled != '_');
 	       ++(*mangled), ++i)
 	    buf[i] = **mangled;
 	  if (**mangled != '_')

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]