Re: [PATCH] disassemble support start,+length format

[Cheng Renquan <crquan@gmail.com>]

On Fri, Apr 9, 2010 at 4:21 PM, Eli Zaretskii <eliz@gnu.org> wrote:
>> -* "disassemble" command with a /r modifier, print the raw instructions
>> -in hex as well as in symbolic form.
>> +* The "disassemble" command with an optional /r modifier, print the raw
>> +instructions in hex as well as in symbolic form; optional /m modifier to
>> +print mixed source+assembly.
>
> Thanks.  But please use the style we use in other NEWS entries.
> Something like "The disassemble command now supports ..." etc.

The previous entries in "Changes in GDB 7.0" already have problems, I
should check,

>
>> +arguments specify a range of addresses, in the form of "start,end" or
>> +"start,+length", stand for [start,end) and [start,start+length), (first
>> +inclusive, second exclusive) to dump.  In that case, the name of the
>> +function is also printed (since there could be several functions in the
>> +given range).
>
> I suggest to rephrase as follows:
>
>  arguments specify a range of addresses to dump, in one of two forms:
>
>  @table @code
>  @item @var{start},@var{end}
>  the addresses from @var{start} (inclusive) to @var{end} (exclusive)
>  @item @var{start},+@var{length}
>  the addresses from @var{start} (inclusive) to
>  @code{@var{start}+@var{length}} (exclusive).
>  @end table
>
>  @noindent
>  When 2 arguments are specified, the name of the function is also
>  printed (since there could be several functions in the given range).
>
> Note that I used @var{start},@var{end}}, instead of using double
> quotes (which is generally a no-no in Texinfo sources).

OK, I really know little about texinfo writing; Thanks,

-- 
Cheng Renquan (程任全), from Singapore