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:*
[[email protected]](http://code.google.com/u/107439649205250398889/)
*Originally owned by:*
[[email protected]](http://code.google.com/u/107439649205250398889/)
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](http://codereview.appspot.com/194100043)
---
Sent from sourceforge.net because [email protected] 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
[email protected]
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
