Re: Change \retrograde to deal with ties and repeat chords/notes (issue 302470043 by address@hidden)

[thomasmorley65]

On 2016/08/16 07:54:50, dak wrote:
> On 2016/08/16 06:23:17, thomasmorley651 wrote:
> > Didn't dive into the code in depth. From my (limited) testings, I'd
say it's
> > very nice, thus:

> I'm really bad at creating test cases.  Do you have any spares left
over from
> your (limited) testings that would make for a regtest?

Nope. Though, I felt challenged and did some more testings and found
some issues.

With this example:

m = \relative g' { <g~ b>4 g8\glissando f <g b>4~ g8\glissando f }

\score {
  \new Voice { \m \retrograde  \m }
  \layout { ragged-right = ##f }
}

One Tie is omitted (without warning), glissandi not printed.


For a regtest perhaps:

motif =
\relative {
  \override TextSpanner.bound-details.left.text = "motif"
  <c' e>4~\startTextSpan  c16\< d^( e f~ f4:32)\!
  <<
    \context Voice = "voice" { <g~ b>4 g8\glissando f\stopTextSpan }
    \\
    { c2 }
  >>
}

\new Voice = "voice" {
  \motif
  \override TextSpanner.bound-details.left.text = "retrograde motif"
  \retrograde  \motif
}

Ofcourse the glissando isn't printed yet, as well. Interestingly a
warning about unterminated tie is issued, though.

https://codereview.appspot.com/302470043/