more: minor fixes to magic()

Read the magic bytes into signed chars instead of vanilla chars in
order to ensure consistent results even on systems whose char type has
no sign.  Eliminate spurious parentheses in return statements.
Correct grammatical errors in comments.

Signed-off-by: James Youngman <jay@gnu.org>

diff --git a/text-utils/more.c b/text-utils/more.c
index dc38ff2a771c1fdcfe9d58dc2c199736bdadc560..f251b279f670846a29cab14eb4e9916c2fb83745 100644 (file)
--- a/text-utils/more.c
+++ b/text-utils/more.c
@@ -531,18 +531,18 @@ checkf (fs, clearfirst)
 /*
  * magic --
  *     check for file magic numbers.  This code would best be shared with
- *     the file(1) program or, perhaps, more should not try and be so smart?
+ *     the file(1) program or, perhaps, more should not try to be so smart.
  */
 static int
 magic(f, fs)
        FILE *f;
        char *fs;
 {
-       char twobytes[2];
+       signed char twobytes[2];
 
        /* don't try to look ahead if the input is unseekable */
        if (fseek(f, 0L, SEEK_SET))
-               return(0);
+               return 0;
 
        if (fread(twobytes, 2, 1, f) == 1) {
                switch(twobytes[0] + (twobytes[1]<<8)) {
@@ -555,11 +555,11 @@ magic(f, fs)
                case 0x457f:            /* simple ELF detection */
                        printf(_("\n******** %s: Not a text file ********\n\n"), fs);
                        (void)fclose(f);
-                       return(1);
+                       return 1;
                }
        }
        (void)fseek(f, 0L, SEEK_SET);           /* rewind() not necessary */
-       return(0);
+       return 0;
 }
 
 /*