This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Remove type prefix for -var-evaluate-expression/functions


> > > the following patch removes 'type prefix' from output of
> > > -var-evaluate-expression applied to objects of function type (not
> > > pointers to functions, but functions).
> > > 
> > > It causes no regression in the testsuite for me.
> > 
> > But I guess a new test would help prevent a regression (in the code) in
> > the future.
> 
> Yes, can you suggest which file should I add this new testcase too?

Well I guess mi-var-cmd.exp.  Please remember that I'm not the maintainer
but just an interested party.  See what Daniel says.  He might want something
for mi2-var-cmd.exp too.


> > > +  if (TYPE_CODE (type) == TYPE_CODE_FUNC)
> > 
> > This condition must always be true here (case TYPE_CODE_FUNC:).
> 
> This chunk is in 'c_value_print', where there's no switch.      

Ah yes!  I misread the patch.

> I guess I'd better send the patch with "-p". It's attached, hopefully it's
> more clear.

> And thanks for the hint about -p!

Hmm...yes, I guess I've proved my own point.

-- 
Nick                                           http://www.inet.net.nz/~nickrob


PS I'm not subscribed to gdb-patches so please include me in your replies.
I think its fairly standard to do this as its more inconvenient to get no
reply, than get two and have to delete one.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]