[PATCH 4/4] [v2] winmm: Support TIME_MIDI position in MIDI stream.
Akihiro Sagawa
sagawa.aki at gmail.com
Wed Jul 11 08:33:25 CDT 2018
In v2,
Remove awkward goto statement, duplicate TIME_MS code fragment instead.
Andrew, as an alternative, do you prefer the following way?
---
redo:
switch (lpMMT->wType) {
: : :
/* fall through to alternative format */
case TIME_MS:
lpMMT->u.ms = midistream_get_playing_position(lpMidiStrm);
break;
default:
lpMMT->wType = TIME_MS;
goto redo;
}
---
Signed-off-by: Akihiro Sagawa <sagawa.aki at gmail.com>
---
dlls/winmm/tests/midi.c | 4 ++--
dlls/winmm/winmm.c | 21 ++++++++++++++++++---
2 files changed, 20 insertions(+), 5 deletions(-)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0004-v2-winmm-Support-TIME_MIDI-position-in-MIDI-stream.patch
Type: text/x-patch
Size: 2605 bytes
Desc: not available
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20180711/4d88a3ff/attachment-0001.bin>
More information about the wine-devel
mailing list