* some warning patches
@ 2004-12-13 5:02 Ben Elliston
2004-12-13 18:57 ` Keith Seitz
0 siblings, 1 reply; 2+ messages in thread
From: Ben Elliston @ 2004-12-13 5:02 UTC (permalink / raw)
To: insight
[-- Attachment #1: Type: text/plain, Size: 2507 bytes --]
This patch should be sufficiently obvious. Perhaps gdbtk_two_elem_cmd
should take a const char * as its second argument, but I've just
followed the tradition set throughout gdbtk-hooks.c. ;-)
Ben
2004-12-13 Ben Elliston <bje@au.ibm.com>
Remove some warnings:
* generic/gdbtk-hooks.c (gdbtk_pre_add_symbol): Cast `name'.
* generic/gdbtk-cmds.c (gdbtk_dis_asm_read_memory): Remove.
Index: generic/gdbtk-hooks.c
===================================================================
RCS file: /home/bje/src-cvs/src/gdb/gdbtk/generic/gdbtk-hooks.c,v
retrieving revision 1.38
diff -u -p -r1.38 gdbtk-hooks.c
--- generic/gdbtk-hooks.c 26 Jun 2004 20:30:51 -0000 1.38
+++ generic/gdbtk-hooks.c 13 Dec 2004 05:00:16 -0000
@@ -597,7 +597,7 @@ gdbtk_load_hash (const char *section, un
static void
gdbtk_pre_add_symbol (const char *name)
{
- gdbtk_two_elem_cmd ("gdbtk_tcl_pre_add_symbol", name);
+ gdbtk_two_elem_cmd ("gdbtk_tcl_pre_add_symbol", (char *) name);
}
/* This hook is called whenever we finish loading a symbol file. */
Index: generic/gdbtk-cmds.c
===================================================================
RCS file: /home/bje/src-cvs/src/gdb/gdbtk/generic/gdbtk-cmds.c,v
retrieving revision 1.84
diff -u -p -r1.84 gdbtk-cmds.c
--- generic/gdbtk-cmds.c 19 Nov 2004 00:19:56 -0000 1.84
+++ generic/gdbtk-cmds.c 13 Dec 2004 05:00:16 -0000
@@ -180,8 +180,6 @@ static int gdb_stop (ClientData, Tcl_Int
static int gdb_target_has_execution_command (ClientData,
Tcl_Interp *, int,
Tcl_Obj * CONST[]);
-static int gdbtk_dis_asm_read_memory (bfd_vma, bfd_byte *, unsigned int,
- disassemble_info *);
static void gdbtk_load_source (ClientData clientData,
struct symtab *symtab,
int start_line, int end_line);
@@ -2075,27 +2073,6 @@ gdb_disassemble_driver (CORE_ADDR low, C
}
return TCL_OK;
-}
-
-/* This is the memory_read_func for gdb_disassemble_driver when we are
- disassembling from the exec file. */
-
-static int
-gdbtk_dis_asm_read_memory (bfd_vma memaddr, bfd_byte *myaddr,
- unsigned int len, disassemble_info *info)
-{
- extern struct target_ops exec_ops;
- int res;
-
- errno = 0;
- res = xfer_memory (memaddr, myaddr, len, 0, 0, &exec_ops);
-
- if (res == len)
- return 0;
- else if (errno == 0)
- return EIO;
- else
- return errno;
}
/* This will be passed to qsort to sort the results of the disassembly */
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: some warning patches
2004-12-13 5:02 some warning patches Ben Elliston
@ 2004-12-13 18:57 ` Keith Seitz
0 siblings, 0 replies; 2+ messages in thread
From: Keith Seitz @ 2004-12-13 18:57 UTC (permalink / raw)
To: Ben Elliston; +Cc: insight
On Sun, 2004-12-12 at 21:03, Ben Elliston wrote:
> This patch should be sufficiently obvious.
Yeah, that is pretty obvious. Please commit.
Keith
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2004-12-13 18:57 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-12-13 5:02 some warning patches Ben Elliston
2004-12-13 18:57 ` Keith Seitz
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).