Rémi Bernon (@rbernon) commented about dlls/dmime/midi.c:
parser->time += event.delta_time; if (event.status == 0xff && event.meta_type == MIDI_META_SET_TEMPO) hr = midi_parser_handle_set_tempo(parser, &event); - else if ((event.status & 0xf0) == MIDI_PROGRAM_CHANGE) + else + { + switch (event.status & 0xf0) + { + case MIDI_NOTE_ON: + case MIDI_NOTE_OFF: hr = midi_parser_handle_note_on_off(parser, &event); break;
```suggestion:-0+0 case MIDI_NOTE_OFF: hr = midi_parser_handle_note_on_off(parser, &event); break; ``` Please use a style consistent across cases (yes, clang-format is bad at it). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5221#note_63497