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: Re: [Patch] Try2: -var-evaluate-expression [-f FORMAT] NAME


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


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