Re: Revised autobeam settings patch (issue1682049)

[n . puttock]

Hi Carl,

LGTM.

The web snippet granados.ly uses beatLength, so will also need emending.

Cheers,
Neil


http://codereview.appspot.com/1682049/diff/35001/36004
File Documentation/notation/rhythms.itely (right):

http://codereview.appspot.com/1682049/diff/35001/36004#newcode1055
Documentation/notation/rhythms.itely:1055: predefined default values for
these values can be found in
for these properties

http://codereview.appspot.com/1682049/diff/35001/36004#newcode1060
Documentation/notation/rhythms.itely:1060: \score{
\score {

http://codereview.appspot.com/1682049/diff/35001/36004#newcode1730
Documentation/notation/rhythms.itely:1730: @funindex beamExceptions
+ beatStructure

http://codereview.appspot.com/1682049/diff/35001/36004#newcode1755
Documentation/notation/rhythms.itely:1755: for the beam type, use it to
determine the valid places where
beam-type ?

http://codereview.appspot.com/1682049/diff/35001/36004#newcode1824
Documentation/notation/rhythms.itely:1824: @emph{complete} exceptions
list.  That is, every exception that should
lists

http://codereview.appspot.com/1682049/diff/35001/36009
File
Documentation/snippets/new/conducting-signs,-measure-grouping-signs.ly
(right):

http://codereview.appspot.com/1682049/diff/35001/36009#newcode16
Documentation/snippets/new/conducting-signs,-measure-grouping-signs.ly:16:
the measure.  @code{time} and @code{set-time-signature} both apply
@code{\time}

http://codereview.appspot.com/1682049/diff/35001/36009#newcode18
Documentation/snippets/new/conducting-signs,-measure-grouping-signs.ly:18:
@code{beatStructure} or @code{baseUnit} that are set in
baseMoment

http://codereview.appspot.com/1682049/diff/35001/36014
File input/regression/auto-beam-beaming-override.ly (right):

http://codereview.appspot.com/1682049/diff/35001/36014#newcode11
input/regression/auto-beam-beaming-override.ly:11: \version "2.13.27"
2.13.28

http://codereview.appspot.com/1682049/diff/35001/36016
File input/regression/beaming-ternary-metrum.ly (right):

http://codereview.appspot.com/1682049/diff/35001/36016#newcode2
input/regression/beaming-ternary-metrum.ly:2: \version "2.13.27"
2.13.28

http://codereview.appspot.com/1682049/diff/35001/36017
File input/regression/les-nereides.ly (right):

http://codereview.appspot.com/1682049/diff/35001/36017#newcode1
input/regression/les-nereides.ly:1: \version "2.13.27"
2.13.28

http://codereview.appspot.com/1682049/diff/35001/36019
File lily/beam-engraver.cc (right):

http://codereview.appspot.com/1682049/diff/35001/36019#newcode309
lily/beam-engraver.cc:309: "baseMoment ",
move to top

http://codereview.appspot.com/1682049/diff/35001/36025
File lily/timing-translator.cc (right):

http://codereview.appspot.com/1682049/diff/35001/36025#newcode62
lily/timing-translator.cc:62: context ()->set_property ("baseMoment",
add to translator doc (+ others missing)

http://codereview.appspot.com/1682049/diff/35001/36026
File ly/bagpipe.ly (right):

http://codereview.appspot.com/1682049/diff/35001/36026#newcode12
ly/bagpipe.ly:12: \version "2.13.27"
2.13.28

http://codereview.appspot.com/1682049/diff/35001/36027
File ly/engraver-init.ly (right):

http://codereview.appspot.com/1682049/diff/35001/36027#newcode19
ly/engraver-init.ly:19: \version "2.13.27"
2.13.28

http://codereview.appspot.com/1682049/diff/35001/36028
File ly/music-functions-init.ly (right):

http://codereview.appspot.com/1682049/diff/35001/36028#newcode21
ly/music-functions-init.ly:21: \version "2.13.27"
2.13.28

http://codereview.appspot.com/1682049/diff/35001/36028#newcode675
ly/music-functions-init.ly:675: (revert-time-signature-setting
time-signature context))
indent

http://codereview.appspot.com/1682049/diff/35001/36030
File scm/auto-beam.scm (right):

http://codereview.appspot.com/1682049/diff/35001/36030#newcode62
scm/auto-beam.scm:62: (not (eq? (member moment beat-structure) #f)))
(pair? (member moment beat-structure))

http://codereview.appspot.com/1682049/diff/35001/36030#newcode127
scm/auto-beam.scm:127: ;; no rule applies, so end at beatLength or
measure end
indent

http://codereview.appspot.com/1682049/diff/35001/36030#newcode129
scm/auto-beam.scm:129: ;; end if measure-pos matches a specified ending
moment
indent

http://codereview.appspot.com/1682049/diff/35001/36037
File scm/time-signature-settings.scm (right):

http://codereview.appspot.com/1682049/diff/35001/36037#newcode253
scm/time-signature-settings.scm:253: (define (revert-property-setting
context property setting)
public?

http://codereview.appspot.com/1682049/diff/35001/36037#newcode270
scm/time-signature-settings.scm:270: time-signature setting . rest)
this indentation is unmaintainable

http://codereview.appspot.com/1682049/diff/35001/36037#newcode273
scm/time-signature-settings.scm:273: (context-spec-music
indent

http://codereview.appspot.com/1682049/diff/35001/36037#newcode287
scm/time-signature-settings.scm:287: time-signature setting 'Score))
move to previous line

http://codereview.appspot.com/1682049/show