Re: Doc: AU - document -dhelp CLI options (issue 5843060)

2012-03-25 Thread graham 

LGTM

http://codereview.appspot.com/5843060/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Doc: AU - document -dhelp CLI options (issue 5843060)

2012-03-20 Thread pkx166h 


http://codereview.appspot.com/5843060/diff/1/Documentation/usage/running.itely
File Documentation/usage/running.itely (right):

http://codereview.appspot.com/5843060/diff/1/Documentation/usage/running.itely#newcode56
Documentation/usage/running.itely:56: * Basic command line options for
lilypond::
On 2012/03/19 02:03:46, Graham Percival wrote:

how about:



Command-line options with --help
Command-line options with -dhelp



?  I'm not totally confident about this idea, though, so don't take it

as

gospel.


Made it simpler - I used 'Basic' and 'Advanced' command line options for
Lilypond

http://codereview.appspot.com/5843060/diff/1/Documentation/usage/running.itely#newcode201
Documentation/usage/running.itely:201: @option{-dsafe}, see @ref{Command
line options using scheme values}
On 2012/03/19 02:03:46, Graham Percival wrote:

@ref should come at the end of a sentence, or at least have some

strong

punctuation after it (like a semicolon).  Or maybe in this case a

parenthesis

makes more sense.


Done.

http://codereview.appspot.com/5843060/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Doc: AU - document -dhelp CLI options (issue 5843060)

2012-03-18 Thread graham 

looks basically good


http://codereview.appspot.com/5843060/diff/1/Documentation/usage/running.itely
File Documentation/usage/running.itely (right):

http://codereview.appspot.com/5843060/diff/1/Documentation/usage/running.itely#newcode56
Documentation/usage/running.itely:56: * Basic command line options for
lilypond::
how about:

Command-line options with --help
Command-line options with -dhelp

?  I'm not totally confident about this idea, though, so don't take it
as gospel.

http://codereview.appspot.com/5843060/diff/1/Documentation/usage/running.itely#newcode201
Documentation/usage/running.itely:201: @option{-dsafe}, see @ref{Command
line options using scheme values}
@ref should come at the end of a sentence, or at least have some strong
punctuation after it (like a semicolon).  Or maybe in this case a
parenthesis makes more sense.

http://codereview.appspot.com/5843060/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Doc: AU - document -dhelp CLI options (issue 5843060)

2012-03-18 Thread pkx166h 

Reviewers: ,

Message:
I will need some advice on any inconsistent/incorrect terms for
variables/symbols/values.

Description:
Doc: AU - document -dhelp CLI options

Issue 2216

Add new @node for the -d[Scheme_val] option.
Renamed old node so that it is distinguishable from the
cli using sceheme variables.

Moved all the original -d[symbol] information over and added
some @ref{} for different options.

Other minor syntax changes and spacing edits on surrounding text.

Please review this at http://codereview.appspot.com/5843060/

Affected files:
  M Documentation/usage/running.itely



___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel