> I'm not top posting.

Hi there!

I noticed an always false check in midi.c.
'track_size' is unsigned and can't be negative.
Should the check look different, or is it redundant?

Best regards,
Nicolas Kaiser

Signed-off-by: Nicolas Kaiser <ni...@nikai.net>
---
 python/midi.c |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/python/midi.c b/python/midi.c
index 187268b..0cc5073 100644
--- a/python/midi.c
+++ b/python/midi.c
@@ -364,9 +364,6 @@ pymidi_parse_track (PyObject *self, PyObject *args)
     return 0;
   debug_print ("clocks_max: %d\n", clocks_max);
 
-  if (track_size < 0)
-    return midi_error (__FUNCTION__,   ": negative track size: ", compat_itoa 
(track_size));
-
   track_end = track + track_size;
   
   return midi_parse_track (&track, track_end, clocks_max);
-- 
1.7.5.rc3


_______________________________________________
bug-lilypond mailing list
bug-lilypond@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-lilypond

Reply via email to