#3497 Accidentals cause cross-voice arpeggio collisions

[Anonymous]

Originally created by: *anonymous

Originally created by: k-ohara5...@oco.net
Originally owned by: mts...@gmail.com

An example from issue 601, which was reported as *working* in contrast to the similar case that fails, now fails.  The arpeggio collides with previous notes.

\new Staff  \with { \consists "Span_arpeggio_engraver" }
\relative c'' {
   \set Staff.connectArpeggios = ##t
   << {[r2]. <ges aes c ges'>4\arpeggio |}
      \\      {\repeat unfold 12 aes,16 <ees aes c>4\arpeggio |}
   >> }

First bad commit is [r7d3d28de0ce6e2f018aff599cecd944d1754fe3c]
Date:   Thu Jan 10 08:54:12 2013 +0100

    Makes all side-positioning based on skylines instead of boxes.
   
issue 2527

[Google Importer]

Originally posted by: mts...@gmail.com

Only marks truly cross-staff arpeggios as cross-staff

http://codereview.appspot.com/13063045

Labels: Patch-new
Owner: mts...@gmail.com
Status: Started

[Google Importer]

Originally posted by: pkx1...@gmail.com

Patchy the autobot says: passes tests.

Labels: -Patch-new Patch-review

[Google Importer]

Originally posted by: pkx1...@gmail.com

Patch on Countdown for August 27th - 06:00 GMT

Labels: -Patch-review Patch-countdown

[Google Importer]

Originally posted by: pkx1...@gmail.com

Patch counted down, please push.

Labels: -Patch-countdown Patch-push

[Google Importer]

Originally posted by: mts...@gmail.com

Pushed as [r0c849c547f242e3aa64032d15b9211a579b75faf].

Labels: Fixed_2_17_26
Status: Fixed

[Google Importer]

Originally posted by: Elu...@gmail.com

(No comment was entered for this change.)

Status: Verified

[Anonymous]

.