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

From: Nick Roberts <nickrob at snap dot net dot nz>
To: "Marc Khouzam" <marc dot khouzam at ericsson dot com>
Cc: "Daniel Jacobowitz" <drow at false dot org>, <gdb-patches at sourceware dot org>
Date: Wed, 6 Feb 2008 10:18:12 +1300
Subject: RE: [Patch] -var-evaluate-expression NAME [FORMAT]
References: <18344.51774.858278.592849@kahikatea.snap.net.nz> <6D19CA8D71C89C43A057926FE0D4ADAA04290E99@ecamlmw720.eamcs.ericsson.se>

 > You don't like my new patch? :-)
 > I feels less risky (and it's cleaner than
 > my orginal one.)

Well it's probably smaller than it looks as there are many formatting changes.
"Don't like" is probably too strong, it just seems a bit intrusive for a
feature that only you may use.  However, I don't object to your patch,...

 > My hesitation with your suggestion is that it
 > seems more risky to forget a case, and that it
 > is not as future-proof because it duplicates
 > some logic in mi-var-cmd and varobj.

...and it probably is safer.

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

References:
- RE: [Patch] -var-evaluate-expression NAME [FORMAT]
  - From: Nick Roberts
- RE: [Patch] -var-evaluate-expression NAME [FORMAT]
  - From: Marc Khouzam

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