[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