Re: D language support

[Mihail Zenkov <mihail dot zenkov at gmail dot com>]

On Wed, 28 Apr 2010 20:09:49 +0300
Eli Zaretskii <eliz@gnu.org> wrote:

> > Date: Wed, 28 Apr 2010 02:53:34 +0300
> > From: Mihail Zenkov <mihail.zenkov@gmail.com>
> > Cc: Mihail Zenkov <mihail.zenkov@gmail.com>, tromey@redhat.com,  gdb-patches@sourceware.org
> > 
> > gdb/NEWS:
> > * D language support.
> >  Support for debugging the D programming language has been
> >  integrated into GDB.
> 
> Suggest to say this instead:
> 
>    GDB now supports debugging programs written in the D programming
>    language.
> 
> > gdb/doc/ChangeLog:
> > 	* gdb.texinfo: Add mention about D language support.
> 
> The ChangeLog entry should include the name(s) of node(s) added to the
> manual.
> 
> > +@cindex D
> > +Support for D is partial.  For information on D, see
> > +@ref{D,,D}.
> 
> Please remove the @cindex entry from here.  You already have one in
> the section where the D support is described; having another identical
> index entry pointing here, where D is not described at all, is not
> useful.  The index entries are for readers who are looking for a
> description of some specific subject, so they should point to the place
> where that subject is described in full.
> 
> > +@value{GDBN} can be used to debug programs written in D and compiled with
> > +GDC, LDC or DMD.
> 
> Suggest to add "compilers" to the end of this sentence.  Also, please
> leave two spaces between sentences.
> 
> > Currently @value{GDBN} supports only one D specific
> > +feature - dynamic arrays.
>            ^
> Please use "---" (3 dashes in a row) to produce an em-dash.
> 
> Okay with those changes.
> 
> Thanks.

Fixed.

Attachment: gdb-100429-d.patch
Description: Binary data