[patch #4610] Consolidated documentation patch

[Benno Schulenberg]

Follow-up Comment #6, patch #4610 (project grep):

This patch and file should go in before a release, in my opinion, as they
describe grep's new colouring possibilities.  I myself don't like the man
page to be structured (preferring to see the options in strict alphabetical
order), but let's wait what the users say.  A few suggested changes to the
wording of the texinfo patch:

or does whatever  -->  or gives whatever / or produces whatever

The general synopsis of a  -->  ...of the

and there can  -->  There can

summarizing these  -->  summarizing the

behaved like @sc{gnu} @command{grep}.  -->  address@hidden's.

minimum size field width  -->  minimum-size field width

no effect unless  -->   no effect unless the

Skip files @emph{and directories}  -->  Skip files

mentioned in the command line  --> mentioned on...

the text terminal that is used  -->  your text terminal


Furthermore, instead of things like "@item mt=01;31", it should say "@item
mt=" and add the default value between parentheses in
subsequent text: "... The default (01;31) is a bold red text..."

I hate the "(`-...' is specified by POSIX)" things.

Not in the patch, but in my opinion mistakenly described:

`[:' represents the open character class followed by a valid character class
name.  -->  `[:' represents the open character class symbol, and should be
followed by a valid character class name.

`:]' represents the close character class followed by a valid character class
name.  -->  `:]' represents the close character class symbol.

There is something wrong with the Concept Index and the Index: the first
wraps to section 2.1.5 (after context), the last to section 5.1 (Fundamental
Structure).  I would expect the first to wrap to the Index, and the last to
wrap to the very start of manual, or to not wrap at all.  They apparently
jump to the first entry in their list when scrolling beyond the end.  Maybe
this is a generic info bug?

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?func=detailitem&item_id=4610>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/