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@gmail.com --- dlls/winmm/tests/midi.c | 4 ++-- dlls/winmm/winmm.c | 21 ++++++++++++++++++--- 2 files changed, 20 insertions(+), 5 deletions(-)