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: [RFA] comment looks wrong...


Daniel Jacobowitz wrote:
On Wed, Oct 28, 2009 at 12:23:11PM -0700, Michael Snyder wrote:
The comment in findvar about extract_*_integer looked wrong to me,
with respect to target vs. host.  If I'm right, this is an improvement,
but if I'm confused, then so are these comments.   ;-/

It's some subject/object fuzziness, I think.


-/* Basic byte-swapping routines.  GDB has needed these for a long time...
-   All extract a target-format integer at ADDR which is LEN bytes long.  */
+/* Basic byte-swapping routines.  GDB has needed these for a long
+   time...  All 'extract' functions extract a host-format integer at
+   ADDR which is LEN bytes long.  */

Which do you extract - the value being read or the value being returned? It means the value being read, which is in target format.


Does that mean you don't favor changing it?



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