From 1012bd0e5fda59fc0a86df4350b206e2b4895ad7 Mon Sep 17 00:00:00 2001 From: Eli Zaretskii Date: Mon, 4 Jun 2001 07:53:08 +0000 Subject: [PATCH] * mips-tdep.c (show_mipsfpu_command): Remove unused variable msg. (mips_set_processor_type_command): Remove unused variable j. (mips_breakpoint_from_pc): Declare breakpoint instruction sequences as unsigned char, to avoid compiler warnings. --- gdb/ChangeLog | 5 +++++ gdb/mips-tdep.c | 27 +++++++++++++++------------ 2 files changed, 20 insertions(+), 12 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 30121c1..0a034b9 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,10 @@ 2001-06-04 Eli Zaretskii + * mips-tdep.c (show_mipsfpu_command): Remove unused variable msg. + (mips_set_processor_type_command): Remove unused variable j. + (mips_breakpoint_from_pc): Declare breakpoint instruction + sequences as unsigned char, to avoid compiler warnings. + * source.c (mod_path, openp): Use HAVE_DOS_BASED_FILE_SYSTEM instead of system-specific define's like _WIN32 and __MSDOS__. Use IS_DIR_SEPARATOR and IS_ABSOLUTE_PATH instead of SLASH_P and diff --git a/gdb/mips-tdep.c b/gdb/mips-tdep.c index e10c9e9..bdb285f 100644 --- a/gdb/mips-tdep.c +++ b/gdb/mips-tdep.c @@ -2193,8 +2193,8 @@ mips_push_arguments (int nargs, don't use float registers for arguments. This duplication of arguments in general registers can't hurt non-MIPS16 functions because those registers are normally skipped. */ - /* MIPS_EABI squeeses a struct that contains a single floating - point value into an FP register instead of pusing it onto the + /* MIPS_EABI squeezes a struct that contains a single floating + point value into an FP register instead of pushing it onto the stack. */ if (fp_register_arg_p (typecode, arg_type) && float_argreg <= MIPS_LAST_FP_ARG_REGNUM) @@ -3279,7 +3279,6 @@ set_mips_command (char *args, int from_tty) static void show_mipsfpu_command (char *args, int from_tty) { - char *msg; char *fpu; switch (MIPS_FPU_TYPE) { @@ -3387,7 +3386,7 @@ mips_show_processor_type_command (char *args, int from_tty) int mips_set_processor_type (char *str) { - int i, j; + int i; if (str == NULL) return 0; @@ -3493,16 +3492,17 @@ mips_breakpoint_from_pc (CORE_ADDR * pcptr, int *lenptr) { if (pc_is_mips16 (*pcptr)) { - static char mips16_big_breakpoint[] = MIPS16_BIG_BREAKPOINT; + static unsigned char mips16_big_breakpoint[] = + MIPS16_BIG_BREAKPOINT; *pcptr = UNMAKE_MIPS16_ADDR (*pcptr); *lenptr = sizeof (mips16_big_breakpoint); return mips16_big_breakpoint; } else { - static char big_breakpoint[] = BIG_BREAKPOINT; - static char pmon_big_breakpoint[] = PMON_BIG_BREAKPOINT; - static char idt_big_breakpoint[] = IDT_BIG_BREAKPOINT; + static unsigned char big_breakpoint[] = BIG_BREAKPOINT; + static unsigned char pmon_big_breakpoint[] = PMON_BIG_BREAKPOINT; + static unsigned char idt_big_breakpoint[] = IDT_BIG_BREAKPOINT; *lenptr = sizeof (big_breakpoint); @@ -3520,16 +3520,19 @@ mips_breakpoint_from_pc (CORE_ADDR * pcptr, int *lenptr) { if (pc_is_mips16 (*pcptr)) { - static char mips16_little_breakpoint[] = MIPS16_LITTLE_BREAKPOINT; + static unsigned char mips16_little_breakpoint[] = + MIPS16_LITTLE_BREAKPOINT; *pcptr = UNMAKE_MIPS16_ADDR (*pcptr); *lenptr = sizeof (mips16_little_breakpoint); return mips16_little_breakpoint; } else { - static char little_breakpoint[] = LITTLE_BREAKPOINT; - static char pmon_little_breakpoint[] = PMON_LITTLE_BREAKPOINT; - static char idt_little_breakpoint[] = IDT_LITTLE_BREAKPOINT; + static unsigned char little_breakpoint[] = LITTLE_BREAKPOINT; + static unsigned char pmon_little_breakpoint[] = + PMON_LITTLE_BREAKPOINT; + static unsigned char idt_little_breakpoint[] = + IDT_LITTLE_BREAKPOINT; *lenptr = sizeof (little_breakpoint); -- 2.7.4