Integrated: 8319598: SMFParser misinterprets interrupted running status
Jan Trukenmüller
duke at openjdk.org
Wed May 1 16:49:02 UTC 2024
On Tue, 7 Nov 2023 19:18:28 GMT, Jan Trukenmüller <duke at 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.
This pull request has now been integrated.
Changeset: 62d5d1e9
Author: Jan Trukenmüller <truj at users.noreply.github.com>
Committer: Phil Race <prr at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/62d5d1e99c118b6ed26e79a2f7247308f8c23310
Stats: 154 lines in 2 files changed: 151 ins; 0 del; 3 mod
8319598: SMFParser misinterprets interrupted running status
Reviewed-by: prr, serb
-------------
PR: https://git.openjdk.org/jdk/pull/16546
More information about the client-libs-dev
mailing list