On Sat, 11 Nov 2023 12:32:15 GMT, Jan Trukenmüller <d...@openjdk.org> wrote:

>> The MIDI file parser misinterprets events without status byte when they 
>> appear directly after a Meta of SysEx event.
>> 
>> For my bugfix I had to decide between two possible solutions:
>> - Strict solution: Throw an InvalidMidiDataException
>> - Tolerant solution: Use the status of the last channel event as running 
>> status
>> 
>> I used the tolerant solution.
>> I will send an email to the list client-libs-dev with my reasons.
>
> Jan Trukenmüller has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   reduced line lengths

Marked as reviewed by serb (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/16546#pullrequestreview-2016467422

Reply via email to