diff options
| author | Patrick Palka <patrick@parcs.ath.cx> | 2015-12-28 22:00:14 +0000 |
|---|---|---|
| committer | Mike Frysinger <vapier@gentoo.org> | 2016-01-05 15:01:28 -0500 |
| commit | 4bec0ef03e91506caf60d8842786b29c2d2ff049 (patch) | |
| tree | 09d0f407174fa9e41d6765ca423a5cbd948e906b /libiberty/ChangeLog | |
| parent | 13b356b2b5c1857490b7911dbbb48e17a3a92b04 (diff) | |
| download | gdb-4bec0ef03e91506caf60d8842786b29c2d2ff049.zip gdb-4bec0ef03e91506caf60d8842786b29c2d2ff049.tar.gz gdb-4bec0ef03e91506caf60d8842786b29c2d2ff049.tar.bz2 | |
libiberty: Tweak the documentation of libiberty's xcrc32 function
In some places the xcrc32 documentation refers to GDB's own crc32
implementation, but GDB no longer has its own crc32 implementation.
It now uses libiberty's xcrc32 throughout. So this patch removes
these references to GDB's now-nonexistent crc32 implementation.
Also, there appears to be a bug in the table-generation program embedded
within the documentation. When the variable "int i" is >= 128, the
computation "i << 24" shifts a one bit into the sign bit (assuming a
32-bit int), which is UB. To avoid this UB, I think it is sufficient to
make the induction variables i and j have type unsigned int. This bug
seems latent, however. I ran the program before and after this change
and the table output is the same.
Diffstat (limited to 'libiberty/ChangeLog')
| -rw-r--r-- | libiberty/ChangeLog | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/libiberty/ChangeLog b/libiberty/ChangeLog index e8fc96a..aa598f2 100644 --- a/libiberty/ChangeLog +++ b/libiberty/ChangeLog @@ -1,3 +1,11 @@ +2015-12-28 Patrick Palka <ppalka@gcc.gnu.org> + + * crc32.c: In the documentation, don't refer to GDB's + now-nonexistent crc32 implementation. In the table-generation + program embedded within the documentation, change the type of + the induction variables i and j from int to unsigned int, to + avoid undefined behavior. + 2015-12-21 Nick Clifton <nickc@redhat.com> PR 66827 |