[m-rev.] for review: Add additional mdb aliases.

Zoltan Somogyi zoltan.somogyi at runbox.com
Fri Aug 9 21:07:10 AEST 2019



On Fri, 9 Aug 2019 19:48:33 +1000, Peter Wang <novalazy at gmail.com> wrote:
> > > +The @samp{step} command can be used to step forward by one or more events.
> > > +As mentioned above, you can also enter an empty line or a number of steps.
> > > ...
> > >  The @samp{quit} command will exit the debugger.
> > >  
> > >  That should be enough to get you started.
> > 
> > Why this addition?
> 
> Which addition?

The large block of paragraphs that explains common commands, NOT the abbreviations.
These started with the the description of the "step" command, and ended just before
the description of "quit", which was already there.
 
> > Aren't these commands already documented elsewhere?
> > Or if this is a new get-started-quickly section, why isn't it labelled as such?
> 
> The section is called "Quick overview".

In that case, why is a full list of the abbreviations there? The "quick overview"
should be *separate* from (a) the comprehensive lists of commands and their options,
and (b) the comprehensive list of aliases. At the most, it should list the aliases
only of the commands it describes. And the structure should avoid introducing aliases
for commands it has not described yet.

Zoltan.


More information about the reviews mailing list