* xcoffread.c (struct coff_symbol): Change c_sclass to unsigned char.

	Remove FIXME comment regarding this.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 004036238fa1d4c2fcfbe2cb479493d7c74490ac..18868a7ec6c2f60ba7583f812bc65eaff1226bb3 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,27 +1,8 @@
-Tue Apr  6 00:24:32 1993  Per Bothner  (bothner@rtl.cygnus.com)
-
-       * ch-exp.y:  Remove ambiguities.  Fix or add support for
-       UNOP_IND, UNOP_CAST, UNOP_ADDR, STRUCOP_STRUCT, OP_TYPE.
-       * ch-typeprint.c (chill_print_type):  Move support
-       for TYPE_CODE_ARRAY and TYPE_CODE_STRING from here ...
-       * ch-typeprint.c (chill_type_print_base): ... here.
-       * ch-typeprint.c (chill_type_print_base):  Improve
-       support for PTR, REF, and FUNC types.
-       * ch-valprint.c:  Prefer H'%x format to 0x%s.
-       * ch-valprint.c (chill_val_print):  Add support for printing
-       TYPE_CODE_REF (return for LOC parameters and variables).
-       Use c_val_print to print random types (including ENUM).
-       * ch-valprint,c (chill_print_value_fields):  Make
-       output look more like Chill tuples.
-       * stabsread.c (read_struct_type):  Factor common code.
-
-Wed Apr  7 11:22:43 1993  Fred Fish  (fnf@cygnus.com)
-
-       * sparclite/Makefile.in:  Add dummy mostlyclean, clean,
-       distclean, and realclean targets.
-
 Wed Apr  7 10:11:22 1993  Jim Kingdon  (kingdon@cygnus.com)
 
+       * xcoffread.c (struct coff_symbol): Change c_sclass to unsigned char.
+       Remove FIXME comment regarding this.
+
        * symfile.h: Change NULL->'\0' in comment (that wasn't a typo).
 
        * xcoffread.c (read_xcoff_symtab): Use E_SYMNMLEN.

diff --git a/gdb/xcoffread.c b/gdb/xcoffread.c
index 6d1606b93b01bef9b24b55264d96799e8436d06d..a8f9e7c0b582decf445c96c0d07885bfebf5756a 100644 (file)
--- a/gdb/xcoffread.c
+++ b/gdb/xcoffread.c
@@ -73,7 +73,7 @@ struct coff_symbol {
   int c_symnum;                /* symbol number of this entry */
   int c_nsyms;         /* 0 if syment only, 1 if syment + auxent */
   long c_value;
-  int c_sclass;
+  unsigned char c_sclass;
   int c_secnum;
   unsigned int c_type;
 };
@@ -1083,13 +1083,7 @@ read_xcoff_symtab (objfile, nsyms)
        symname_alloced = 1;
       }
       cs->c_value = symbol->n_value;
-      /* n_sclass is signed (FIXME), so we had better not mask off any
-        high bits it contains, since the values we will be comparing
-        it to are also signed (FIXME).  Defined in <coff/internal.h>.
-        At this point (3Jun92, gnu@cygnus.com) I think the fix is to
-        make the fields and values unsigned chars, but changing the next
-        line is a simple patch late in the release cycle, for now.  */
-      cs->c_sclass = symbol->n_sclass /* & 0xff */;
+      cs->c_sclass = symbol->n_sclass;
       cs->c_secnum = symbol->n_scnum;
       cs->c_type = (unsigned)symbol->n_type;