Re: add-symbol-file parsing

[Richard Sharpe <rsharpe@richardsharpe.com>]

On Tue, 3 Dec 2002, Daniel Jacobowitz wrote:

> On Fri, Nov 29, 2002 at 11:54:48AM +0100, Mark Santcroos wrote:
> > Unlike the documentation says the 'address' argument to 'add-symbol-file'
> > is not parsed as an expression but directly as an address.
> > 
> > The following commands ...
> > 
> > (gdb) set $text_addr = 0xdeadc0de
> > (gdb) add-symbol-file /dir/symbol.file $text_addr
> > 
> > .. will end up setting the address to 0 by strtoul down in the code path.
> > 
> > The above will therefor not work. Is there another way I can achieve this?
> > 
> > If this is not possible (in another way) will patches be accepted to make
> > this an expression?
> 
> The problem is, add-symbol-file takes more than a $text_addr.  It also
> takes a series of -s SECNAME $sec_addr.  We could:
> 
>  - split the command line at -s options and parse anything before the
>    next -s option as an expression, assuming people will not foolishly
>    use -s in their expression;
>  - allow only convenience variables
> 
> Option 2 may be the way to go... What do others think about allowing
> integer-or-convenience-variable there?

I would love to be able to use convenience variables for the file name and 
the text_addr. I started looking at it in 5.2.1, but quickly got bogged 
down in a twisty maze of code.

Regards
-----
Richard Sharpe, rsharpe[at]ns.aus.com, rsharpe[at]samba.org, 
sharpe[at]ethereal.com, http://www.richardsharpe.com