Re: [PATCH v11 0/8] MR4982: Load and play MIDI

[Rémi Bernon (＠rbernon)]

On Tue Feb  6 17:44:25 2024 +0000, Yuxuan Shui wrote:
> you mean splitting out `dmime: Implement getting/setting TempoParam for
> MIDI tracks.`?

I mean opening a separate MR (or updating this one), with only the first 4-5 commits that introduce the midi parser, stubs and parse the basic events.

Then, in a another MR, add tests for MIDI tempo handling, parse the tempo events and implement the tempo parameters.

Having smaller merge requests makes it easier to review and eventually approve, as the amount of changes is reduced.