[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Revised autobeam settings patch (issue1682049)
From: |
n . puttock |
Subject: |
Re: Revised autobeam settings patch (issue1682049) |
Date: |
Sun, 11 Jul 2010 23:31:50 +0000 |
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
- Re: Revised autobeam settings patch (issue1682049), (continued)
- Re: Revised autobeam settings patch (issue1682049), Carl . D . Sorensen, 2010/07/05
- Re: Revised autobeam settings patch (issue1682049), Carl . D . Sorensen, 2010/07/05
- Re: Revised autobeam settings patch (issue1682049), n . puttock, 2010/07/06
- Re: Revised autobeam settings patch (issue1682049), Carl . D . Sorensen, 2010/07/07
- Re: Revised autobeam settings patch (issue1682049), Carl . D . Sorensen, 2010/07/07
- Re: Revised autobeam settings patch (issue1682049), Carl . D . Sorensen, 2010/07/07
- Re: Revised autobeam settings patch (issue1682049), n . puttock, 2010/07/07
- Re: Revised autobeam settings patch (issue1682049), n . puttock, 2010/07/07
- Re: Revised autobeam settings patch (issue1682049),
n . puttock <=
- Re: Revised autobeam settings patch (issue1682049), Carl . D . Sorensen, 2010/07/12
- Re: Revised autobeam settings patch (issue1682049), Carl . D . Sorensen, 2010/07/13
- Re: Revised autobeam settings patch (issue1682049), Carl . D . Sorensen, 2010/07/13
- Re: Revised autobeam settings patch (issue1682049), n . puttock, 2010/07/29