[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5132 LM 2.4.1: fix errors in variable definitions

[Auto mailings of changes to Lily Issues]

Doc: fix errors in variable definitions - issue #5132

http://codereview.appspot.com/325720043

---

[issues:#5132] LM 2.4.1: fix errors in variable definitions

Status: Started
Created: Wed May 03, 2017 03:50 PM UTC by Federico Bruni
Last Updated: Fri May 05, 2017 12:38 PM UTC
Owner: nobody

Reported almost 10 months ago:
http://lists.gnu.org/archive/html/bug-lilypond/2016-07/msg00072.html

I found a bug in the Learning Manual:
http://lilypond.org/doc/v2.19/Documentation/learning/organizing-pieces-with-variables.html

The example suggested at the end of that page doesn't work:

1. paperheight should be renamed to paper-height

2. name variable should be renamed to something else (myName), as it triggers this error:

warning: identifier name is a keyword: name'

1. width variable triggers this error:
   error: unknown escaped string: \cm'

I suggested to change it to:

myWidth = 60
myName = "Wendy"
aFivePaper = \paper { #(set-paper-size "a5") }

\paper {
  \aFivePaper
  line-width = \myWidth
}

{
  c'4^\myName
}

We may want to add that default unit of line-width value is \mm and that changing the unit in the variable definition is not allowed. See also this comment:
http://lists.gnu.org/archive/html/bug-lilypond/2016-07/msg00078.html

---

Sent from sourceforge.net because address@hidden is subscribed to https://sourceforge.net/p/testlilyissues/issues/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot

_______________________________________________
Testlilyissues-auto mailing list
address@hidden
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto