Warning fixes.

diff --git a/binutils/ChangeLog b/binutils/ChangeLog
index cbb116528c406e4d4b620550425135789a81c877..abbda4faeb80c8e39eee3f5afe5e860dbf7245dd 100644 (file)
--- a/binutils/ChangeLog
+++ b/binutils/ChangeLog
@@ -1,3 +1,18 @@
+2000-07-10  David Huggins-Daines  <dhd@linuxcare.com>
+
+       * stabs.c (parse_stab_range_type): Cast to proper type to fix
+       warning.
+
+2000-07-10  Ryan Bradetich  <rbradetich@uswest.net>
+
+       * readelf.c (get_dynamic_data): Add casts and use unsigned chars
+       to fix warnings.
+       (process_symbol_table): Ditto.
+       (procesS_extended_line_op): Ditto.
+       (display_debug_lines): Ditto.
+       (display_debug_pubnames): Ditto.
+       (read_and_display_attr): Ditto.
+
 2000-07-09  Nick Clifton  <nickc@cygnus.com>
 
        * MAINTAINERS: Add several more port specific maintainers.

diff --git a/binutils/readelf.c b/binutils/readelf.c
index 64207d95e4e16a5afae5129ee1fefabc218e3237..83cba0b496f0647a90fb3b0317e27a07236e945b 100644 (file)
--- a/binutils/readelf.c
+++ b/binutils/readelf.c
@@ -4230,10 +4230,10 @@ get_dynamic_data (file, number)
      FILE *       file;
      unsigned int number;
 {
-  char * e_data;
+  unsigned char * e_data;
   int *  i_data;
 
-  e_data = (char *) malloc (number * 4);
+  e_data = (unsigned char *) malloc (number * 4);
 
   if (e_data == NULL)
     {
@@ -4270,8 +4270,8 @@ process_symbol_table (file)
      FILE * file;
 {
   Elf32_Internal_Shdr *   section;
-  char   nb [4];
-  char   nc [4];
+  unsigned char   nb [4];
+  unsigned char   nc [4];
   int    nbuckets = 0;
   int    nchains = 0;
   int *  buckets = NULL;
@@ -4885,7 +4885,7 @@ process_extended_line_op (data, is_stmt, pointer_size)
 
       printf (_("   %d\t"), ++ state_machine_regs.last_file_entry);
       name = data;
-      data += strlen (data) + 1;
+      data += strlen ((char *) data) + 1;
       printf (_("%lu\t"), read_leb128 (data, & bytes_read, 0));
       data += bytes_read;
       printf (_("%lu\t"), read_leb128 (data, & bytes_read, 0));
@@ -4990,7 +4990,7 @@ display_debug_lines (section, start, file)
            {
              printf (_("  %s\n"), data);
 
-             data += strlen (data) + 1;
+             data += strlen ((char *) data) + 1;
            }
        }
 
@@ -5007,13 +5007,13 @@ display_debug_lines (section, start, file)
 
          while (* data != 0)
            {
-             char * name;
+             unsigned char * name;
              int bytes_read;
 
              printf (_("  %d\t"), ++ state_machine_regs.last_file_entry);
              name = data;
 
-             data += strlen (data) + 1;
+             data += strlen ((char *) data) + 1;
 
              printf (_("%lu\t"), read_leb128 (data, & bytes_read, 0));
              data += bytes_read;
@@ -5183,7 +5183,7 @@ display_debug_pubnames (section, start, file)
            {
              data += 4;
              printf ("    %ld\t\t%s\n", offset, data);
-             data += strlen (data) + 1;
+             data += strlen ((char *) data) + 1;
            }
        }
       while (offset != 0);
@@ -6008,7 +6008,7 @@ read_and_display_attr (attribute, form, data, cu_offset, pointer_size)
 
     case DW_FORM_string:
       printf (" %s", data);
-      data += strlen (data) + 1;
+      data += strlen ((char *) data) + 1;
       break;
 
     case DW_FORM_block:

diff --git a/binutils/stabs.c b/binutils/stabs.c
index 5f600354257e3b11550f096d8a5db51a6773bea5..90f7666690eed48ec432c8a419c1039dec99cd78 100644 (file)
--- a/binutils/stabs.c
+++ b/binutils/stabs.c
@@ -1,5 +1,5 @@
 /* stabs.c -- Parse stabs debugging information
-   Copyright (C) 1995, 1996, 1997, 1998, 1999 Free Software Foundation, Inc.
+   Copyright (C) 1995, 96, 97, 98, 99, 2000 Free Software Foundation, Inc.
    Written by Ian Lance Taylor <ian@cygnus.com>.
 
    This file is part of GNU Binutils.
@@ -1822,7 +1822,7 @@ parse_stab_range_type (dhandle, info, typename, pp, typenums)
          else if (n3 == (bfd_signed_vma) 0xffffffff)
            return debug_make_int_type (dhandle, 4, true);
 #ifdef BFD64
-         else if (n3 == ((((bfd_vma) 0xffffffff) << 32) | 0xffffffff))
+         else if (n3 == ((((bfd_signed_vma) 0xffffffff) << 32) | 0xffffffff))
            return debug_make_int_type (dhandle, 8, true);
 #endif
        }