Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(879)

Issue 164096: Chord repetition: \relative mode (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
14 years, 4 months ago by nicolas.sceaux
Modified:
13 years ago
Reviewers:
dak
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Chord repetition: \relative mode Introduce a new RepeatedChord music type, which element property is the copied chord (as produced by the chord repetition function). Define a relative callback for repeated chords, which avoids octaviation of repeated chords, by modifying the first note octave before applying the octave relativization.

Patch Set 1 #

Patch Set 2 : Chord repetition: chord memorization function #

Patch Set 3 : Chord repetition: memorize <..> chords only, fix \relative mode #

Unified diffs Side-by-side diffs Delta from patch set Stats (+133 lines, -23 lines) Patch
M Documentation/notation/simultaneous.itely View 1 chunk +8 lines, -0 lines 0 comments Download
A input/regression/chord-repetition-relative.ly View 1 chunk +13 lines, -0 lines 0 comments Download
M lily/include/music-sequence.hh View 1 chunk +1 line, -0 lines 0 comments Download
M lily/music-sequence.cc View 1 2 1 chunk +55 lines, -0 lines 0 comments Download
M lily/parser.yy View 2 3 chunks +4 lines, -6 lines 0 comments Download
M ly/chord-repetition-init.ly View 2 2 chunks +37 lines, -14 lines 0 comments Download
scm/define-music-properties.scm View 1 chunk +2 lines, -0 lines 0 comments Download
scm/define-music-types.scm View 1 chunk +9 lines, -0 lines 0 comments Download
scm/ly-syntax-constructors.scm View 1 2 1 chunk +4 lines, -3 lines 0 comments Download

Messages

Total messages: 4
nicolas.sceaux
Hi, Here is a patch to make chord repetition work in \relative mode. As I'm ...
14 years, 4 months ago (2009-12-03 10:56:31 UTC) #1
nicolas.sceaux
Hi, I've uploaded a second patch set, which introduces a chord memorization function. When dealing ...
14 years, 4 months ago (2009-12-06 17:23:13 UTC) #2
dak
On 2009/12/06 17:23:13, nicolas.sceaux wrote: > When dealing with a simple_chord_element (simple notes, rests, etc) ...
14 years, 4 months ago (2009-12-06 19:30:03 UTC) #3
nicolas.sceaux
14 years, 4 months ago (2009-12-06 21:11:25 UTC) #4
Le 6 déc. 2009 à 20:30, dak@gnu.org a écrit :

> I think it is the wrong way to go if notes can only be guessed if you
> know what user-settable memorization function is in effect.  We should
> not sacrifice readability for the sake of saving a few keystrokes.

This is a valid objection, but which kind of music elements should be
copied by q, then? (which was my question in the first place)
If people think that e.g. chords only should be memorized, then I'm fine
with that.  The implementation will be easier.

On the other hand... which ratio of users might ever change the behavior
of chord repetition memorization?  I don't really think that readability
might suffer, as long as the default is the most convenient.  For instance,
users can also completely change the note names, this is possible being 
given the interface for setting note names, but the fact that it is possible
does not by itself cause readability problems.  It could be even adviced to
change the repetition shortcut when changing the memorization function, to
avoid readability problems (or at least warn the reader that something
unusual happens there).

As a LilyPond user who maintains more than 100'000 lines of LilyPond code,
what I cherish the most about LilyPond is its extensability.  Is it really
needed here, I don't know yet.

Nicolas

Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b