From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Jacobowitz To: Joel Brobecker Cc: gdb-patches@sources.redhat.com Subject: Re: [RFA/RFC] New command: ``start'' Date: Fri, 21 May 2004 00:26:00 -0000 Message-id: <20040521002645.GA17522@nevyn.them.org> References: <20040518214712.GD4036@nevyn.them.org> <20040518222740.GZ10684@gnat.com> <20040518224131.GA6026@nevyn.them.org> <20040519153615.GD10684@gnat.com> <20040519154155.GA8567@nevyn.them.org> <20040520010145.GQ10684@gnat.com> <20040520134600.GA11705@nevyn.them.org> <20040520160259.GV10684@gnat.com> <20040520171404.GA3880@nevyn.them.org> <20040520221214.GC10684@gnat.com> X-SW-Source: 2004-05/msg00621.html On Thu, May 20, 2004 at 03:12:14PM -0700, Joel Brobecker wrote: > OK, here is a new patch for the "start" command. I don't need to modify > main_name() right now, so I left it out of this patch. > > When Ada support is activated, then the next step will be to add the > new-executable notification, and add a call to the ada_main_name > routine in main_name(). I am attaching a patch to symtab.c for > illustration. This looks pretty good. I have one question: > + /* Check that there is a program to debug. Some languages such as Ada > + need to search inside the program symbols for the location where to > + put the temporary breakpoint before starting. */ > + if (!have_full_symbols () && !have_partial_symbols ()) > + error ("No symbol table loaded. Use the \"file\" command."); Shouldn't you accept have_minimal_symbols here? -- Daniel Jacobowitz