remove pointless assignment
authorTrevor Saunders <tbsaunde+binutils@tbsaunde.org>
Wed, 13 Apr 2016 09:03:22 +0000 (05:03 -0400)
committerTrevor Saunders <tbsaunde+binutils@tbsaunde.org>
Sat, 16 Apr 2016 01:20:36 +0000 (21:20 -0400)
Presumably this was supposed to be regname[sizeof (regname) - 1] but was typoed
to regname[sizeof (rename) - 1].  However that should be unnecessary because
sprintf should null terminate.  As is this assignment is invalid ISO C because
rename refers to the function rename (), and sizeof on functions is undefined.
In GNU C C the size of functions is 1 so the expression is the same as
regname[0].  The following call to sprintf () clearly will over right that, so
the statement either has no effect or is invalid.  Given that it seems safe to
just remove it.  While we are there correct the size of regname, and switch
from snprintf to sprintf since we know the exact length of the result.

gas/ChangeLog:

2016-04-15  Trevor Saunders  <tbsaunde+binutils@tbsaunde.org>

* config/tc-mips.c (md_begin): Remove useless assignment.

gas/ChangeLog
gas/config/tc-mips.c

index bfa91e72a18575b2ee888a7a446cd286d3656fd6..f9b5afeb56c237fb0d4cb9f84f13e175d99e4f55 100644 (file)
@@ -1,3 +1,7 @@
+2016-04-15  Trevor Saunders  <tbsaunde+binutils@tbsaunde.org>
+
+       * config/tc-mips.c (md_begin): Remove useless assignment.
+
 2016-04-15  H.J. Lu  <hongjiu.lu@intel.com>
 
        * Makefile.in: Regenerated with automake 1.11.6.
index 118b91dfe61e090f2b3f982b982591d0e5ec705e..bfc639e154e9fc1313f985c73fcb2fa30185abb1 100644 (file)
@@ -3601,21 +3601,20 @@ md_begin (void)
 
   for (i = 0; i < 32; i++)
     {
-      char regname[7];
+      char regname[6];
 
       /* R5900 VU0 floating-point register.  */
-      regname[sizeof (rename) - 1] = 0;
-      snprintf (regname, sizeof (regname) - 1, "$vf%d", i);
+      sprintf (regname, "$vf%d", i);
       symbol_table_insert (symbol_new (regname, reg_section,
                                       RTYPE_VF | i, &zero_address_frag));
 
       /* R5900 VU0 integer register.  */
-      snprintf (regname, sizeof (regname) - 1, "$vi%d", i);
+      sprintf (regname, "$vi%d", i);
       symbol_table_insert (symbol_new (regname, reg_section,
                                       RTYPE_VI | i, &zero_address_frag));
 
       /* MSA register.  */
-      snprintf (regname, sizeof (regname) - 1, "$w%d", i);
+      sprintf (regname, "$w%d", i);
       symbol_table_insert (symbol_new (regname, reg_section,
                                       RTYPE_MSA | i, &zero_address_frag));
     }