Bugs item #1911415, was opened at 2008-03-10 20:38 Message generated for change (Settings changed) made by claudiusmaximus You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=478070&aid=1911415...

Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: externals Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: ClaudiusMaximus (claudiusmaximus)

Assigned to: Martin Peach (mrpeach)

Summary: mrpeach/midifile writes invalid midi files

Initial Comment: The midifile external by Martin Peach writes invalid MIDI files that have a nasty habit of making other software crash. The problem is in the header:

00000000 4d 54 68 64 00 00 00 06 00 00 00 01 00 00 4d 54 |MThd..........MT|

Which should be:

00000000 4d 54 68 64 00 00 00 06 00 00 00 01 ** ** 4d 54 |MThd..........MT|

Where **** is a valid MIDI file "time code" value, for example, 0x0060 for 96 ticks (I used hexedit to change it to this value and it mostly worked for what I was doing).

It would be good if midifile.c and midifile-help.pd could both be updated to:

1) disallow 0x0000 in the header 2) allow setting of the time code value in various ways (MIDI files support at least 'ticks' and 'SMPTE') 3) document and explain how to set the time code value

Thanks

----------------------------------------------------------------------

You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=478070&aid=1911415...