bfd/elf32-arc.c:1537]: (warning) Redundant assignment

2018-02-11 Thread David Binderman
Hello there, bfd/elf32-arc.c:1537]: (warning) Redundant assignment of 'rel->r_offset' to itself. Source code is rel->r_offset = rel->r_offset; Suggest code rework. Regards David Binderman ___ bug-binutils mailin

bfd/elf32-nds32.c:9693]: (warning) Redundant assignment

2018-02-11 Thread David Binderman
Hello there, bfd/elf32-nds32.c:9693]: (warning) Redundant assignment of 'irel->r_addend' to itself. Source code is rel->r_offset = rel->r_offset; Suggest code rework. Regards David Binderman ___ bug-binutils mailin

src/gdb/remote.c:7724: missing use of va_end ?

2015-11-09 Thread David Binderman
__, _("Too long remote packet."));   if (putpkt (rs->buf) < 0)     error (_("Communication problem with target."));   rs->buf[0] = '\0';   getpkt (&rs->buf, &rs-&g

src/gold/arm.cc:4513: de-allocation then use ?

2015-11-09 Thread David Binderman
Hello there, [src/gold/arm.cc:4513]: (error) Returning/dereferencing 'buffer' after it is deallocated / released Source code is   delete[] buffer;   return std::string(buffer); Suggest code rework. Regards David

tc-v850.c:1663: possible bad if condition ?

2015-03-02 Thread David Binderman
else if (strcmp (arg, "4byte-align") == 0)     v850_e_flags &= ~ EF_RH850_DATA_ALIGN8; The last two conditions will never be true. Suggest code rework. Regards David Binderman ___ bug-binutils mailing list bug-binutils@gnu.org https://lists.gnu.org/mailman/listinfo/bug-binutils

binutils-2.21.53.0.2/gas bug report

2011-10-23 Thread David Binderman
case 0xe8: There looks to be a missing break in there. 2. [binutils-2.21.53.0.2/gas/config/tc-mn10200.c:563]: (warning) Redundant assignment of "opcode" in switch Duplicate. Regards David Binderman