public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* (no subject)
@ 2022-05-18 15:29 Nick Clifton
  0 siblings, 0 replies; only message in thread
From: Nick Clifton @ 2022-05-18 15:29 UTC (permalink / raw)
  To: binutils

Hi Guys,

  Compiling GOLD with Clang-14 produces a compile time warning about the
  current_byte variable being unused in int_encoding.cc:
  get_length_as_unsigned_LEB_128(), so I am checking in the obvious fix
  below.

Cheers
  Nick

gold/ChangeLog
2022-05-18  Nick Clifton  <nickc@redhat.com>

	* int_encoding.cc (get_length_as_unsigned_LEB_128): Remove
	current_length variable.
  
diff --git a/gold/int_encoding.cc b/gold/int_encoding.cc
index d27b2af4c21..196e0b3b509 100644
--- a/gold/int_encoding.cc
+++ b/gold/int_encoding.cc
@@ -119,12 +119,7 @@ get_length_as_unsigned_LEB_128(uint64_t value)
   size_t length = 0;
   do
     {
-      unsigned char current_byte = value & 0x7f;
       value >>= 7;
-      if (value != 0)
-        {
-          current_byte |= 0x80;
-        }
       length++;
     }
   while (value != 0);


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2022-05-18 15:29 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-18 15:29 Nick Clifton

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).