This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch 2/2] Wrap-up expression support for DFP.
- From: Thiago Jung Bauermann <bauerman at br dot ibm dot com>
- To: Eli Zaretskii <eliz at gnu dot org>
- Cc: gdb-patches at sourceware dot org
- Date: Wed, 02 Jan 2008 14:34:37 -0200
- Subject: Re: [patch 2/2] Wrap-up expression support for DFP.
- References: <20071220054926.148275471@br.ibm.com> <20071220055107.194393592@br.ibm.com> <uzlw4hw5f.fsf@gnu.org> <1198816387.12907.62.camel@localhost.localdomain> <uejd5yb6c.fsf@gnu.org>
On Sat, 2007-12-29 at 13:47 +0200, Eli Zaretskii wrote:
> > From: Thiago Jung Bauermann <bauerman@br.ibm.com>
> > Cc: gdb-patches@sourceware.org
> > Date: Fri, 28 Dec 2007 02:33:07 -0200
> >
> > I couldn't find an appropriate place to put these in the manual, so I
> > went ahead and created a new sub-section, under "Examining Data". What
> > do you think?
>
> I think a better place is in the "C and C++" section. These are C/C++
> data types, aren't they?
>
> > +There are two encodings in use, depending on the architecture: BID (Binary
> > +Integer Decimal) for x86 and x86-64, and DPD (Densely Packed Decimal) for
> > +PowerPC. GDB will use the appropriate encoding for the configured target.
>
> PLease use "@value{GDBN}" instead of a literal "GDB".
>
> > +Because of a limitation in libdecnumber, the library used by @value{GDBN}
>
> "libdecnumber" is a file name, so it should have the @file markup.
>
> > +point computations, error checking in decimal float operations ignore underflow,
>
> "ignores".
>
> Other than these comments, the patch is okay to go in (as a subsection
> of the "C and C++" section.
Ok, changed the patch to abide to all observations above. I will commit
it when the code patch is approved as well. Thanks for your review.
--
[]'s
Thiago Jung Bauermann
Software Engineer
IBM Linux Technology Center