* ar.c (usage): Fix C conformance issue.
authorNick Clifton <nickc@redhat.com>
Thu, 18 Jul 2013 16:12:35 +0000 (16:12 +0000)
committerNick Clifton <nickc@redhat.com>
Thu, 18 Jul 2013 16:12:35 +0000 (16:12 +0000)
* config/tc-i386-intel.c (i386_intel_operand): Fixed signed vs
unsigned comparison.

binutils/ChangeLog
binutils/ar.c
gas/ChangeLog
gas/config/tc-i386-intel.c

index 700d03b..8f0451a 100644 (file)
@@ -1,3 +1,7 @@
+2013-07-18  Jim Thomas  <thomas@cfht.hawaii.edu>
+
+       * ar.c (usage): Fix C conformance issue.
+
 2013-07-18  Nick Clifton  <nickc@redhat.com>
 
        * doc/binutils.texi (nm, objdump): Remove bogus links to STABS
index c424038..af5694d 100644 (file)
@@ -243,8 +243,6 @@ usage (int help)
 {
   FILE *s;
 
-  s = help ? stdout : stderr;
-
 #if BFD_SUPPORTS_PLUGINS
   /* xgettext:c-format */
   const char *command_line
@@ -257,6 +255,8 @@ usage (int help)
     = _("Usage: %s [emulation options] [-]{dmpqrstx}[abcDfilMNoPsSTuvV]"
        " [member-name] [count] archive-file file...\n");
 #endif
+  s = help ? stdout : stderr;
+
   fprintf (s, command_line, program_name);
 
   /* xgettext:c-format */
index 87908d7..04229f9 100644 (file)
@@ -1,3 +1,8 @@
+2013-07-18  Jim Thomas  <thomas@cfht.hawaii.edu>
+
+       * config/tc-i386-intel.c (i386_intel_operand): Fixed signed vs
+       unsigned comparison.
+
 2013-07-18  Sandeep Kumar Singh  <Sandeep.Singh2@kpitcummins.com>
 
         * config/rx-defs.h: Add macros for RX100, RX200, RX600, and
index aefdc2a..9080861 100644 (file)
@@ -792,7 +792,7 @@ i386_intel_operand (char *operand_string, int got_a_float)
           || intel_state.is_mem)
     {
       /* Memory operand.  */
-      if (i.mem_operands
+      if ((int) i.mem_operands
          >= 2 - !current_templates->start->opcode_modifier.isstring)
        {
          /* Handle