|
From: | Auto mailings of changes to Lily Issues |
Subject: | [Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5121 midi2ly: cryptic comment |
Date: | Sun, 23 Apr 2017 13:12:48 +0000 |
[issues:#5121] midi2ly: cryptic comment
Status: Accepted
Created: Sun Apr 16, 2017 02:40 PM UTC by Colin Campbell
Last Updated: Sun Apr 16, 2017 02:40 PM UTC
Owner: nobody
Conversation between Christopher Heckman and Simon Albrecht, regarding midi2ly:
(3) At line 677, there's a cryptic comment
urg, this will barf at meter changes
Is there a brief explanation of the problem available?
Well, apparently this particular code isn’t prepared to deal with meter
changes in the input. What exactly is unclear about that?Best, Simon
First question: Is the exact problem known, or did midi2ly crash (or
whatever -- that's another question), and someone said, "Oops, better
not do that"? Or ... did someone recognize that the bug would occur,
and put this here instead of trying to fix it?
I haven’t been involved in coding this at all, but to me the comment sounds like the author saw this limitation but didn’t want to fix it right away.
Are there meter changes where midi2ly does not crash?
That would be up to the person tackling the limitation to figure out.
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
[Prev in Thread] | Current Thread | [Next in Thread] |