[PR #13258] Hide the use of MIDI behind velocity #29455

Closed
opened 2026-01-31 09:35:00 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/13258

State: closed
Merged: Yes


  • conhost requires an additional dependency in Windows, which might
    cause us trouble in WPG
  • Terminal requires an additional package dependency, which will
    cause us trouble in WPG (since GmDls is about 3MB)

I chose to scope the feature checks to MidiOut directly, as I wanted to
keep the delay behavior in MidiAudio::PlayNote. This is negotiable.

References #13252

**Original Pull Request:** https://github.com/microsoft/terminal/pull/13258 **State:** closed **Merged:** Yes --- * conhost requires an additional dependency in Windows, which might cause us trouble in WPG * Terminal requires an additional *package* dependency, which *will* cause us trouble in WPG (since GmDls is about 3MB) I chose to scope the feature checks to MidiOut directly, as I wanted to keep the delay behavior in MidiAudio::PlayNote. This is negotiable. References #13252
claunia added the pull-request label 2026-01-31 09:35:00 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#29455