lilypond-auto
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Lilypond-auto] Issue 3673 in lilypond: Patch: Make make-relative able t


From: lilypond
Subject: [Lilypond-auto] Issue 3673 in lilypond: Patch: Make make-relative able to deal with music rather than just pitches
Date: Fri, 22 Nov 2013 12:54:20 +0000

Status: Started
Owner: address@hidden
Labels: Type-Enhancement Patch-new

New issue 3673 by address@hidden: Patch: Make make-relative able to deal with music rather than just pitches
http://code.google.com/p/lilypond/issues/detail?id=3673

Make make-relative able to deal with music rather than just pitches

I'm leaning towards letting this slip into 2.18 still.  The reason is
that make-relative in its current form is only useful for a quite
limited number of cases, can't be easily coded as a subcase of the new
behavior, and the convert-ly rule only covers the most important use
cases.

Due to its limited usefulness, it has not seen much exposure on the
mailing lists and is likely not in use much, so retaining the function
name and obliterating the old, less useful behavior, makes sense
before 2.18.  Afterwards, this would be harder.

Consists of commits (reverse order):

Add regtest for music argument semantics of make-relative macro


Run convert-ly on input/regression/make-relative.ly


convert-ly rule for new make-relative semantics accepting music arguments

This only works when the "reference pitch" is the first or last of
the list.

Make make-relative able to deal with music rather than just pitches

The old behavior of make-relative was too cumbersome to use in many
cases.

This variant allows, for example:

withOctave =
  (ly:music?)
  (make-relative
   (music) music
   #{ \context Bottom << $music \transpose c c' $music >> #}))

\relative
\new Staff \withOctave {
  \partial 4. c'8 e g |
  c2 e,4 g |
  c,8 c' b a <g d'> <f c'> <e b'> <d a'> |
  <c g'>1 | \bar "|."
}

http://codereview.appspot.com/30890043

--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings



reply via email to

[Prev in Thread] Current Thread [Next in Thread]