[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] Ticket 4252 discussion

[Auto mailings of changes to Lily Issues via Testlilyissues-auto]

I see "the motivation for this patch is to eliminate redundant code". I am currently trying to deal with fixing the Volta_repeat_iterator. Seminal features here are that its actions are relevant for all of the Timing context (not just the context the construct may appear in), and that every alternative starts with the same meter and current chord as the first alternative. It would appear that this isn't easily achieved outside of a specialized iterator, so the redundancy argument is likely a bit shaky concerning at least some of the semantics. It's not going to see a change for 2.20 but I am not clear about how to proceed here in the long run.

---

[issues:#4252] Patch: Remove Unfolded_repeat_iterator

Status: Verified
Labels: Fixed_2_19_16
Created: Thu Jan 08, 2015 08:54 PM UTC by Anonymous
Last Updated: Mon Mar 02, 2015 04:28 AM UTC
Owner: nobody

Originally created by: *anonymous

Originally created by: address@hidden
Originally owned by: address@hidden

Remove Unfolded_repeat_iterator and rely on Sequential_iterator with a customized elements-callback factored out of unfold-repeats-fully.

http://codereview.appspot.com/194100043

---

Sent from sourceforge.net because address@hidden is subscribed to https://sourceforge.net/p/testlilyissues/issues/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot

_______________________________________________
Testlilyissues-auto mailing list
address@hidden
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto