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