From c54e2ec1422ff0ea450229edea966ce994db408c Mon Sep 17 00:00:00 2001 From: Alan Modra Date: Wed, 6 Mar 2013 13:40:51 +0000 Subject: * strings.c (get_char): Dispense with buf[]. Instead shift chars into big-endian value and byte-swap later if little-endian. Don't EOF check value read from object. --- binutils/ChangeLog | 6 ++++++ binutils/strings.c | 25 ++++++------------------- 2 files changed, 12 insertions(+), 19 deletions(-) (limited to 'binutils') diff --git a/binutils/ChangeLog b/binutils/ChangeLog index 1278b1e..bc04889 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,3 +1,9 @@ +2013-03-07 Alan Modra + + * strings.c (get_char): Dispense with buf[]. Instead shift + chars into big-endian value and byte-swap later if + little-endian. Don't EOF check value read from object. + 2013-03-05 Corinna Vinschen * configure.in: Build DLL tools on x86_64-*-cygwin* as well. diff --git a/binutils/strings.c b/binutils/strings.c index 3eed63a..d591630 100644 --- a/binutils/strings.c +++ b/binutils/strings.c @@ -455,8 +455,7 @@ static long get_char (FILE *stream, file_ptr *address, int *magiccount, char **magic) { int c, i; - long r = EOF; - unsigned char buf[4]; + long r = 0; for (i = 0; i < encoding_bytes; i++) { @@ -484,34 +483,22 @@ get_char (FILE *stream, file_ptr *address, int *magiccount, char **magic) } (*address)++; - buf[i] = c; + r = (r << 8) | (c & 0xff); } switch (encoding) { - case 'S': - case 's': - r = buf[0]; - break; - case 'b': - r = (buf[0] << 8) | buf[1]; + default: break; case 'l': - r = buf[0] | (buf[1] << 8); - break; - case 'B': - r = ((long) buf[0] << 24) | ((long) buf[1] << 16) | - ((long) buf[2] << 8) | buf[3]; + r = ((r & 0xff) << 8) | ((r & 0xff00) >> 8); break; case 'L': - r = buf[0] | ((long) buf[1] << 8) | ((long) buf[2] << 16) | - ((long) buf[3] << 24); + r = (((r & 0xff) << 24) | ((r & 0xff00) << 8) + | ((r & 0xff0000) >> 8) | ((r & 0xff000000) >> 24)); break; } - if (r == EOF) - return 0; - return r; } -- cgit v1.1