newlandsvalley / purescript-midi

MIDI and Web MIDI support
MIT License
10 stars 1 forks source link

File and event parsing is subtly different #6

Closed newlandsvalley closed 6 years ago

newlandsvalley commented 6 years ago

In that SysEx messages appear to have a different format within a file from that in an event stream. From p135 in the MIDI spec:

For instance, the transmitted message F0 43 12 00 07 F7 would be stored in a MIDI file as F0 05 43 12 00 07 F7.

Again discovered by @rhofour against the Elm implementation. Unfortunately this means that it will require separate implementations of all event parsing.

newlandsvalley commented 6 years ago

Also, running status messages are disallowed in web-midi generated streams (which is all we claim to support), Fixed by abe1d288f5d08bd1819c7b2341804da612d0e825.