RE: Re: [Patch] Try2: -var-evaluate-expression [-f FORMAT] NAME

Nick Roberts
2008-04-02T21:26:15Z

Marc Khouzam writes:
>
> >         * varobj.h (varobj_get_formatted_value): Declare.
> > >         * varobj.c (my_value_of_variable): Added format parameter.
> > >         (cplus_value_of_variable): Likewise.
> > >         (java_value_of_variable): Likewise.
> > >         (*(value_of_variable)): Likewise.
> >
> > Is '*(value_of_variable)' really a name of a function? :-)
>
> After Daniel's comment, I'm not too sure what you guys do
> for function pointers that are members....
> so I removed this Changelog entry. :-)

It still seems to be there.

(struct language_specific): Add format parameter to function member
*value_of_variable.

> > > +/* Parse a string argument into a format value.  */
> >
> > As meta-remark, can you pass the "-p" option to diff when generating patches,
> > so that function names are present right in the patch?
>
>
> Sounds good but I'm not sure how to tell eclipse to do that...
> I'll keep looking, but until then, please forgive my patch since I ran
> it without

Put "diff -p" in your ~/.cvsrc?

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