Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 18 May 2011 21:06:20 +0000 (UTC)
From:      Ben Laurie <benl@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r222084 - head/contrib/gperf/src
Message-ID:  <201105182106.p4IL6KkE008657@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: benl
Date: Wed May 18 21:06:20 2011
New Revision: 222084
URL: http://svn.freebsd.org/changeset/base/222084

Log:
  Fix clang warnings.
  
  Approved by:	philip (mentor)

Modified:
  head/contrib/gperf/src/gen-perf.cc
  head/contrib/gperf/src/key-list.cc
  head/contrib/gperf/src/options.cc

Modified: head/contrib/gperf/src/gen-perf.cc
==============================================================================
--- head/contrib/gperf/src/gen-perf.cc	Wed May 18 21:04:29 2011	(r222083)
+++ head/contrib/gperf/src/gen-perf.cc	Wed May 18 21:06:20 2011	(r222084)
@@ -246,7 +246,7 @@ Gen_Perf::change (List_Node *prior, List
       {
         if (option[DEBUG])
           {
-            fprintf (stderr, " by changing asso_value['%c'] (char #%d) to %d\n",
+            fprintf (stderr, " by changing asso_value['%c'] (char #%zd) to %d\n",
                      *p, p - union_set + 1, asso_values[(unsigned char)(*p)]);
             fflush (stderr);
           }

Modified: head/contrib/gperf/src/key-list.cc
==============================================================================
--- head/contrib/gperf/src/key-list.cc	Wed May 18 21:04:29 2011	(r222083)
+++ head/contrib/gperf/src/key-list.cc	Wed May 18 21:06:20 2011	(r222084)
@@ -497,8 +497,8 @@ Key_List::merge (List_Node *list1, List_
           *resultp = list1;
           break;
         }
-      if (occurrence_sort && list1->occurrence < list2->occurrence
-          || hash_sort && list1->hash_value > list2->hash_value)
+      if ((occurrence_sort && list1->occurrence < list2->occurrence)
+	    || (hash_sort && list1->hash_value > list2->hash_value))
         {
           *resultp = list2;
           resultp = &list2->next; list2 = list1; list1 = *resultp;
@@ -1035,17 +1035,16 @@ Key_List::output_hash_function (void)
   if (option[CPLUSPLUS])
     printf ("%s::", option.get_class_name ());
   printf ("%s ", option.get_hash_name ());
-  printf (option[KRC] ?
-                 "(str, len)\n"
-            "     register char *str;\n"
-            "     register unsigned int len;\n" :
-          option[C] ?
-                 "(str, len)\n"
-            "     register const char *str;\n"
-            "     register unsigned int len;\n" :
-          option[ANSIC] | option[CPLUSPLUS] ?
-                 "(register const char *str, register unsigned int len)\n" :
-          "");
+  if (option[KRC] || option[C] || option [ANSIC] || option[CPLUSPLUS])
+    printf (option[KRC] ?
+	      "(str, len)\n"
+              "     register char *str;\n"
+              "     register unsigned int len;\n" :
+	    option[C] ?
+	      "(str, len)\n"
+              "     register const char *str;\n"
+              "     register unsigned int len;\n" :
+	      "(register const char *str, register unsigned int len)\n");
 
   /* Note that when the hash function is called, it has already been verified
      that  min_key_len <= len <= max_key_len. */
@@ -1442,7 +1441,7 @@ Key_List::output_lookup_array (void)
 
           if (option[DEBUG])
             fprintf (stderr,
-                     "dup_ptr[%d]: hash_value = %d, index = %d, count = %d\n",
+                     "dup_ptr[%zd]: hash_value = %d, index = %d, count = %d\n",
                      dup_ptr - duplicates,
                      dup_ptr->hash_value, dup_ptr->index, dup_ptr->count);
 
@@ -1986,17 +1985,16 @@ Key_List::output_lookup_function (void)
   if (option[CPLUSPLUS])
     printf ("%s::", option.get_class_name ());
   printf ("%s ", option.get_function_name ());
-  printf (option[KRC] ?
-                 "(str, len)\n"
-            "     register char *str;\n"
-            "     register unsigned int len;\n" :
-          option[C] ?
-                 "(str, len)\n"
-            "     register const char *str;\n"
-            "     register unsigned int len;\n" :
-          option[ANSIC] | option[CPLUSPLUS] ?
-                 "(register const char *str, register unsigned int len)\n" :
-          "");
+  if (option[KRC] || option[C] || option[ANSIC] || option[CPLUSPLUS])
+    printf (option[KRC] ?
+	      "(str, len)\n"
+              "     register char *str;\n"
+              "     register unsigned int len;\n" :
+	    option[C] ?
+	      "(str, len)\n"
+              "     register const char *str;\n"
+              "     register unsigned int len;\n" :
+	    "(register const char *str, register unsigned int len)\n");
 
   /* Output the function's body. */
   printf ("{\n");

Modified: head/contrib/gperf/src/options.cc
==============================================================================
--- head/contrib/gperf/src/options.cc	Wed May 18 21:04:29 2011	(r222083)
+++ head/contrib/gperf/src/options.cc	Wed May 18 21:06:20 2011	(r222084)
@@ -237,7 +237,7 @@ Options::print_options (void)
         {
           putchar (*arg);
           arg++;
-          if (*arg >= 'A' && *arg <= 'Z' || *arg >= 'a' && *arg <= 'z')
+          if ((*arg >= 'A' && *arg <= 'Z') || (*arg >= 'a' && *arg <= 'z'))
             {
               putchar (*arg);
               arg++;



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