New patch set uploaded. Thanks for the review, Keith, and for trying to break the code. I appreciate the thorough testing.
I've tried to put it through a torture test of my own, and as long as I keep things rational, it appears to work. I can "break" it by starting a tuplet at a weird time, but I'm not sure it's broken, given the weird timing in the measures that break it. http://codereview.appspot.com/5495076/diff/1004/lily/beaming-pattern.cc File lily/beaming-pattern.cc (right): http://codereview.appspot.com/5495076/diff/1004/lily/beaming-pattern.cc#newcode174 lily/beaming-pattern.cc:174: int tuplet_count = (int) factor.den (); Yes, tuplet_count should have been the denominator. Somehow I made a mistake earlier using it as the numerator, then copied the mistake here. I've renamed the variable to make it clearer. http://codereview.appspot.com/5495076/ _______________________________________________ lilypond-devel mailing list [email protected] https://lists.gnu.org/mailman/listinfo/lilypond-devel
