diff options
author | Darkvater <Darkvater@openttd.org> | 2007-02-23 12:56:10 +0000 |
---|---|---|
committer | Darkvater <Darkvater@openttd.org> | 2007-02-23 12:56:10 +0000 |
commit | 9fa035f4676a0f2d185f235164af6dfbe0dd038e (patch) | |
tree | 8281000501fad69bd62fdfaf32ebf1cf2d050a9a /src/music/win32_m.cpp | |
parent | 4160d85cd2571604c615500124ba8077fc26bfd2 (diff) | |
download | openttd-9fa035f4676a0f2d185f235164af6dfbe0dd038e.tar.xz |
(svn r8860) -Cleanup: some style changes, proper #endif comments, variable initialisation, WINCE ifdef and a vsprintf to vsnprintf change.
Diffstat (limited to 'src/music/win32_m.cpp')
-rw-r--r-- | src/music/win32_m.cpp | 20 |
1 files changed, 6 insertions, 14 deletions
diff --git a/src/music/win32_m.cpp b/src/music/win32_m.cpp index 36b79b30b..1096584d9 100644 --- a/src/music/win32_m.cpp +++ b/src/music/win32_m.cpp @@ -1,7 +1,6 @@ /* $Id$ */ #include "../stdafx.h" -#include "../openttd.h" #include "win32_m.h" #include <windows.h> #include <mmsystem.h> @@ -50,7 +49,7 @@ static MCIERROR CDECL MidiSendCommand(const char* cmd, ...) char buf[512]; va_start(va, cmd); - vsprintf(buf, cmd, va); + vsnprintf(buf, lengthof(buf), cmd, va); va_end(va); return mciSendStringA(buf, NULL, 0, 0); } @@ -58,12 +57,9 @@ static MCIERROR CDECL MidiSendCommand(const char* cmd, ...) static bool MidiIntPlaySong(const char *filename) { MidiSendCommand("close all"); - if (MidiSendCommand("open \"%s\" type sequencer alias song", filename) != 0) - return false; + if (MidiSendCommand("open \"%s\" type sequencer alias song", filename) != 0) return false; - if (MidiSendCommand("play song from 0") != 0) - return false; - return true; + return MidiSendCommand("play song from 0") == 0; } static void MidiIntStopSong(void) @@ -104,9 +100,7 @@ static DWORD WINAPI MidiThread(LPVOID arg) s[0] = '\0'; // Delay somewhat in case we don't manage to play. - if (!_midi.playing) { - Sleep(5000); - } + if (!_midi.playing) Sleep(5000); } if (_midi.stop_song && _midi.playing) { @@ -115,8 +109,7 @@ static DWORD WINAPI MidiThread(LPVOID arg) MidiIntStopSong(); } - if (_midi.playing && !MidiIntIsSongPlaying()) - _midi.playing = false; + if (_midi.playing && !MidiIntIsSongPlaying()) _midi.playing = false; WaitForMultipleObjects(1, &_midi.wait_obj, FALSE, 1000); } while (!_midi.terminate); @@ -148,8 +141,7 @@ static const char *Win32MidiStart(const char * const *parm) } } - if (CreateThread(NULL, 8192, MidiThread, 0, 0, &threadId) == NULL) - return "Failed to create thread"; + if (CreateThread(NULL, 8192, MidiThread, 0, 0, &threadId) == NULL) return "Failed to create thread"; return NULL; } |