summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authortron <tron@openttd.org>2005-03-09 16:56:26 +0000
committertron <tron@openttd.org>2005-03-09 16:56:26 +0000
commit11ba413c4a30077d00747568cca1fd667c8ab9e2 (patch)
tree02b8d1e68c6c3fa94f02cb66b6c2a6f9e1218c1d
parent6e1e4f1dc23115b33e78e15e0b94af7eb87e582a (diff)
downloadopenttd-11ba413c4a30077d00747568cca1fd667c8ab9e2.tar.xz
(svn r1972) Several cleanups and fix some latent bugs
-rw-r--r--win32.c330
1 files changed, 191 insertions, 139 deletions
diff --git a/win32.c b/win32.c
index a5620e1d6..460e7a618 100644
--- a/win32.c
+++ b/win32.c
@@ -49,7 +49,7 @@ extern const HalMusicDriver _dmusic_midi_driver;
static void MakePalette(void)
{
LOGPALETTE *pal;
- int i;
+ uint i;
byte *b;
pal = alloca(sizeof(LOGPALETTE) + (256-1) * sizeof(PALETTEENTRY));
@@ -57,7 +57,7 @@ static void MakePalette(void)
pal->palVersion = 0x300;
pal->palNumEntries = 256;
- for(i=0,b=_cur_palette; i!=256;i++,b+=3) {
+ for (i = 0, b = _cur_palette; i != 256; i++, b += 3) {
pal->palPalEntry[i].peRed = b[0];
pal->palPalEntry[i].peGreen = b[1];
pal->palPalEntry[i].peBlue = b[2];
@@ -75,7 +75,7 @@ static void UpdatePalette(HDC dc, uint start, uint count)
uint i;
byte *b;
- for(i=0,b = _cur_palette + start*3; i!=count; i++,b+=3) {
+ for (i = 0, b = _cur_palette + start * 3; i != count; i++, b += 3) {
rgb[i].rgbRed = b[0];
rgb[i].rgbGreen = b[1];
rgb[i].rgbBlue = b[2];
@@ -102,8 +102,8 @@ typedef struct {
byte map_to;
} VkMapping;
-#define AS(x,z) {x,0,z}
-#define AM(x,y,z,w) {x,y-x,z}
+#define AS(x, z) {x, 0, z}
+#define AM(x, y, z, w) {x, y - x, z}
#ifndef VK_OEM_3
#define VK_OEM_3 0xC0
@@ -161,7 +161,10 @@ static bool AllocateDibSection(int w, int h);
static void ClientSizeChanged(int w, int h)
{
- if (_wnd.double_size) { w >>= 1; h >>= 1; }
+ if (_wnd.double_size) {
+ w /= 2;
+ h /= 2;
+ }
// allocate new dib section of the new size
if (AllocateDibSection(w, h)) {
@@ -182,7 +185,7 @@ void DoExitSave(void);
static LRESULT CALLBACK WndProcGdi(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
{
- switch(msg) {
+ switch (msg) {
case WM_PAINT: {
PAINTSTRUCT ps;
HDC dc,dc2;
@@ -226,8 +229,8 @@ static LRESULT CALLBACK WndProcGdi(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lP
if (_game_mode == GM_MENU) { // do not ask to quit on the main screen
_exit_game = true;
} else if (_patches.autosave_on_exit) {
- DoExitSave();
- _exit_game = true;
+ DoExitSave();
+ _exit_game = true;
} else
AskExitGame();
@@ -261,8 +264,8 @@ static LRESULT CALLBACK WndProcGdi(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lP
POINT pt;
if (_wnd.double_size) {
- x >>= 1;
- y >>= 1;
+ x /= 2;
+ y /= 2;
}
if (_cursor.fix_at) {
@@ -308,10 +311,10 @@ static LRESULT CALLBACK WndProcGdi(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lP
_pressed_key = w | MapWindowsKey(wParam) << 16;
- if( scancode == 41 )
+ if (scancode == 41)
_pressed_key = w | WKC_BACKQUOTE << 16;
- if ((_pressed_key>>16) == ('D' | WKC_CTRL) && !_wnd.fullscreen) {
+ if ((_pressed_key >> 16) == ('D' | WKC_CTRL) && !_wnd.fullscreen) {
_double_size ^= 1;
_wnd.double_size = _double_size;
ClientSizeChanged(_wnd.width, _wnd.height);
@@ -321,7 +324,7 @@ static LRESULT CALLBACK WndProcGdi(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lP
case WM_SYSKEYDOWN: /* user presses F10 or Alt, both activating the title-menu */
- switch(wParam) {
+ switch (wParam) {
case VK_RETURN: /* Full Screen */
MakeWindow(!_wnd.fullscreen);
return 0;
@@ -352,10 +355,16 @@ static LRESULT CALLBACK WndProcGdi(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lP
w = r->right - r->left - (r2.right - r2.left);
h = r->bottom - r->top - (r2.bottom - r2.top);
- if (_wnd.double_size) { w >>= 1; h >>= 1; }
+ if (_wnd.double_size) {
+ w /= 2;
+ h /= 2;
+ }
w = clamp(w, 64, MAX_SCREEN_WIDTH);
h = clamp(h, 64, MAX_SCREEN_HEIGHT);
- if (_wnd.double_size) { w <<= 1; h <<= 1; }
+ if (_wnd.double_size) {
+ w *= 2;
+ h *= 2;
+ }
SetRect(&r2, 0, 0, w, h);
AdjustWindowRect(&r2, GetWindowLong(hwnd, GWL_STYLE), FALSE);
@@ -405,6 +414,7 @@ static LRESULT CALLBACK WndProcGdi(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lP
case WM_MOUSEWHEEL: {
int delta = GET_WHEEL_DELTA_WPARAM(wParam);
+
if (delta < 0) {
_cursor.wheel++;
} else if (delta > 0) {
@@ -452,7 +462,7 @@ static void MakeWindow(bool full_screen)
_wnd.double_size = _double_size && !full_screen;
// recreate window?
- if ((full_screen|_wnd.fullscreen) && _wnd.main_wnd) {
+ if ((full_screen || _wnd.fullscreen) && _wnd.main_wnd) {
DestroyWindow(_wnd.main_wnd);
_wnd.main_wnd = 0;
}
@@ -469,9 +479,10 @@ static void MakeWindow(bool full_screen)
}
settings.dmPelsWidth = _wnd.width_org;
settings.dmPelsHeight = _wnd.height_org;
- if ((settings.dmDisplayFrequency = _display_hz) != 0)
+ settings.dmDisplayFrequency = _display_hz;
+ if (settings.dmDisplayFrequency != 0)
settings.dmFields |= DM_DISPLAYFREQUENCY;
- if ( !ChangeDisplaySettings(&settings, CDS_FULLSCREEN) == DISP_CHANGE_SUCCESSFUL ) {
+ if (ChangeDisplaySettings(&settings, CDS_FULLSCREEN) != DISP_CHANGE_SUCCESSFUL) {
MakeWindow(false);
return;
}
@@ -485,7 +496,8 @@ static void MakeWindow(bool full_screen)
uint style;
int x, y, w, h;
- if ((_wnd.fullscreen=full_screen) != false) {
+ _wnd.fullscreen = full_screen;
+ if (_wnd.fullscreen) {
style = WS_POPUP | WS_VISIBLE;
SetRect(&r, 0, 0, _wnd.width_org, _wnd.height_org);
} else {
@@ -496,15 +508,16 @@ static void MakeWindow(bool full_screen)
AdjustWindowRect(&r, style, FALSE);
w = r.right - r.left;
h = r.bottom - r.top;
- x = (GetSystemMetrics(SM_CXSCREEN)-w)>>1;
- y = (GetSystemMetrics(SM_CYSCREEN)-h)>>1;
+ x = (GetSystemMetrics(SM_CXSCREEN) - w) / 2;
+ y = (GetSystemMetrics(SM_CYSCREEN) - h) / 2;
if (_wnd.main_wnd) {
SetWindowPos(_wnd.main_wnd, 0, x, y, w, h, SWP_NOACTIVATE | SWP_NOOWNERZORDER | SWP_NOZORDER);
} else {
char Windowtitle[50] = "OpenTTD ";
- // also show revision number/release in window title
- strncat(Windowtitle, _openttd_revision, sizeof(Windowtitle)-(strlen(Windowtitle) + 1));
+
+ snprintf(Windowtitle, lengthof(Windowtitle), "OpenTTD %s",
+ _openttd_revision);
_wnd.main_wnd = CreateWindow("TTD", Windowtitle, style, x, y, w, h, 0, 0, _inst, 0);
if (_wnd.main_wnd == NULL)
@@ -540,7 +553,7 @@ static bool AllocateDibSection(int w, int h)
if (_wnd.double_size) {
w = (w + 3) & ~0x3;
- _wnd.alloced_bits = _wnd.buffer_bits = (byte*)malloc(w * h);
+ _wnd.alloced_bits = _wnd.buffer_bits = malloc(w * h);
w *= 2;
h *= 2;
}
@@ -588,18 +601,21 @@ static void FindResolutions(void)
while (EnumDisplaySettings(NULL, i++, &dm)) {
if (dm.dmBitsPerPel == 8 &&
- IS_INT_INSIDE(dm.dmPelsWidth, 640, MAX_SCREEN_WIDTH+1) &&
- IS_INT_INSIDE(dm.dmPelsHeight, 480, MAX_SCREEN_HEIGHT+1) &&
- (n == 0 || _resolutions[n-1][0] != dm.dmPelsWidth || _resolutions[n-1][1] != dm.dmPelsHeight)) {
+ IS_INT_INSIDE(dm.dmPelsWidth, 640, MAX_SCREEN_WIDTH + 1) &&
+ IS_INT_INSIDE(dm.dmPelsHeight, 480, MAX_SCREEN_HEIGHT + 1) && (
+ n == 0 ||
+ _resolutions[n - 1][0] != dm.dmPelsWidth ||
+ _resolutions[n - 1][1] != dm.dmPelsHeight
+ )) {
_resolutions[n][0] = dm.dmPelsWidth;
_resolutions[n][1] = dm.dmPelsHeight;
- if (++n == sizeof(_resolutions) / (sizeof(uint16)*2)) break;
+ if (++n == lengthof(_resolutions)) break;
}
}
if (n == 0) {
memcpy(_resolutions, default_resolutions, sizeof(default_resolutions));
- n = 6;
+ n = lengthof(default_resolutions);
}
_num_resolutions = n;
@@ -631,9 +647,7 @@ static const char *Win32GdiStart(const char * const *parm)
static void Win32GdiStop(void)
{
- if ( _wnd.fullscreen ) {
- ChangeDisplaySettings(NULL, 0);
- }
+ if (_wnd.fullscreen) ChangeDisplaySettings(NULL, 0);
MyShowCursor(true);
DeleteObject(_wnd.gdi_palette);
DeleteObject(_wnd.dib_sect);
@@ -646,25 +660,28 @@ static void filter(int left, int top, int width, int height)
uint p = _screen.pitch;
byte *s = (byte*)_wnd.buffer_bits + top * p + left;
byte *d = (byte*)_wnd.bitmap_bits + top * p * 4 + left * 2;
- int i;
- while (height) {
- for(i=0; i!=width; i++) {
- d[i*2] = d[i*2+1] = d[i*2+p*2] = d[i*2+1+p*2] = s[i];
+ for (; height > 0; height--) {
+ int i;
+
+ for (i = 0; i != width; i++) {
+ d[i * 2] = d[i * 2 + 1] = d[i * 2 + p * 2] = d[i * 2 + 1 + p * 2] = s[i];
}
s += p;
d += p * 4;
- height--;
}
}
static void Win32GdiMakeDirty(int left, int top, int width, int height)
{
- RECT r = {left, top, left+width, top+height};
+ RECT r = { left, top, left + width, top + height };
+
if (_wnd.double_size) {
filter(left, top, width, height);
- //filter(0, 0, 640, 480);
- r.left *= 2;r.top *= 2;r.right *= 2;r.bottom *= 2;
+ r.left *= 2;
+ r.top *= 2;
+ r.right *= 2;
+ r.bottom *= 2;
}
InvalidateRect(_wnd.main_wnd, &r, FALSE);
}
@@ -699,7 +716,9 @@ static int Win32GdiMainLoop(void)
#endif
/* Disable speeding up game with ALT+TAB (if syskey is pressed, the
* real key is in the upper 16 bits (see WM_SYSKEYDOWN in WndProcGdi()) */
- if (((_pressed_key>>16) & WKC_TAB) && !_networking && _game_mode != GM_MENU) _fast_forward |= 2;
+ if ((_pressed_key >> 16) & WKC_TAB && !_networking &&
+ _game_mode != GM_MENU)
+ _fast_forward |= 2;
} else if (_fast_forward & 2)
_fast_forward = 0;
@@ -766,7 +785,7 @@ const HalVideoDriver _win32_video_driver = {
* WIN32 MIDI PLAYER
**********************/
-struct {
+static struct {
bool stop_song;
bool terminate;
bool playing;
@@ -787,7 +806,7 @@ static void Win32MidiStopSong(void)
{
if (_midi.playing) {
_midi.stop_song = true;
- _midi.start_song[0] = 0;
+ _midi.start_song[0] = '\0';
SetEvent(_midi.wait_obj);
}
}
@@ -806,6 +825,7 @@ static void Win32MidiSetVolume(byte vol)
static long CDECL MidiSendCommand(const char *cmd, ...) {
va_list va;
char buf[512];
+
va_start(va, cmd);
vsprintf(buf, cmd, va);
va_end(va);
@@ -831,7 +851,7 @@ static void MidiIntStopSong(void)
static void MidiIntSetVolume(int vol)
{
uint v = (vol * 65535 / 127);
- midiOutSetVolume((HMIDIOUT)-1, v + (v<<16));
+ midiOutSetVolume((HMIDIOUT)-1, v + (v << 16));
}
static bool MidiIntIsSongPlaying(void)
@@ -843,27 +863,30 @@ static bool MidiIntIsSongPlaying(void)
static DWORD WINAPI MidiThread(LPVOID arg)
{
- char *s;
- int vol;
-
_midi.wait_obj = CreateEvent(NULL, FALSE, FALSE, NULL);
do {
- if ((vol=_midi.new_vol) != -1) {
+ char *s;
+ int vol;
+
+ vol = _midi.new_vol;
+ if (vol != -1) {
_midi.new_vol = -1;
MidiIntSetVolume(vol);
-
}
- if ((s=_midi.start_song)[0]) {
+
+ s = _midi.start_song;
+ if (s[0] != '\0') {
_midi.playing = MidiIntPlaySong(s);
- s[0] = 0;
+ s[0] = '\0';
// Delay somewhat in case we don't manage to play.
if (!_midi.playing) {
Sleep(5000);
}
}
- if (_midi.stop_song != false && _midi.playing) {
+
+ if (_midi.stop_song && _midi.playing) {
_midi.stop_song = false;
_midi.playing = false;
MidiIntStopSong();
@@ -882,6 +905,7 @@ static DWORD WINAPI MidiThread(LPVOID arg)
static const char *Win32MidiStart(const char * const *parm)
{
DWORD threadId;
+
memset(&_midi, 0, sizeof(_midi));
_midi.new_vol = -1;
CreateThread(NULL, 8192, MidiThread, 0, 0, &threadId);
@@ -910,32 +934,37 @@ static WAVEHDR _wave_hdr[2];
static int _bufsize;
static void PrepareHeader(WAVEHDR *hdr)
{
- hdr->dwBufferLength = _bufsize*4;
+ hdr->dwBufferLength = _bufsize * 4;
hdr->dwFlags = 0;
- hdr->lpData = malloc(_bufsize*4);
- if (hdr->lpData == NULL || waveOutPrepareHeader(_waveout, hdr, sizeof(WAVEHDR)) != MMSYSERR_NOERROR)
+ hdr->lpData = malloc(_bufsize * 4);
+ if (hdr->lpData == NULL ||
+ waveOutPrepareHeader(_waveout, hdr, sizeof(WAVEHDR)) != MMSYSERR_NOERROR)
error("waveOutPrepareHeader failed");
}
static void FillHeaders(void)
{
WAVEHDR *hdr;
- for(hdr=_wave_hdr; hdr != endof(_wave_hdr); hdr++) {
+
+ for (hdr = _wave_hdr; hdr != endof(_wave_hdr); hdr++) {
if (!(hdr->dwFlags & WHDR_INQUEUE)) {
- MxMixSamples(_mixer, hdr->lpData, hdr->dwBufferLength >> 2);
+ MxMixSamples(_mixer, hdr->lpData, hdr->dwBufferLength / 4);
if (waveOutWrite(_waveout, hdr, sizeof(WAVEHDR)) != MMSYSERR_NOERROR)
error("waveOutWrite failed");
}
}
}
-static void CALLBACK waveOutProc(HWAVEOUT hwo, UINT uMsg, DWORD dwInstance, DWORD dwParam1, DWORD dwParam2)
+static void CALLBACK waveOutProc(HWAVEOUT hwo, UINT uMsg, DWORD dwInstance,
+ DWORD dwParam1, DWORD dwParam2)
{
- switch(uMsg) {
- case WOM_DONE:
- if (_waveout)
- FillHeaders();
- break;
+ switch (uMsg) {
+ case WOM_DONE:
+ if (_waveout) FillHeaders();
+ break;
+
+ default:
+ break;
}
}
@@ -963,6 +992,7 @@ static const char *Win32SoundStart(const char * const *parm)
static void Win32SoundStop(void)
{
HWAVEOUT waveout = _waveout;
+
_waveout = NULL;
waveOutReset(waveout);
waveOutUnprepareHeader(waveout, &_wave_hdr[0], sizeof(WAVEHDR));
@@ -981,13 +1011,13 @@ bool LoadLibraryList(void **proc, const char *dll)
HMODULE lib;
void *p;
- while (*dll) {
+ while (*dll != '\0') {
lib = LoadLibrary(dll);
if (lib == NULL)
return false;
while (true) {
- while(*dll++);
- if (!*dll)
+ while(*dll++ != '\0');
+ if (*dll == '\0')
break;
p = GetProcAddress(lib, dll);
if (p == NULL)
@@ -1036,22 +1066,22 @@ static void MakeCRCTable(uint32 *table) {
_crc_table = table;
- for (i=0; i!=256; i++) {
+ for (i = 0; i != 256; i++) {
crc = i;
- for (j=8; j!=0; j--) {
+ for (j = 8; j != 0; j--) {
if (crc & 1)
crc = (crc >> 1) ^ poly;
else
- crc>>=1;
+ crc >>= 1;
}
table[i] = crc;
}
}
static uint32 CalcCRC(byte *data, uint size, uint32 crc) {
- do {
- crc = ((crc>>8) & 0x00FFFFFF) ^ _crc_table[ (crc^(*data++)) & 0xFF ];
- } while (--size);
+ for (; size > 0; size--) {
+ crc = ((crc >> 8) & 0x00FFFFFF) ^ _crc_table[(crc ^ *data++) & 0xFF];
+ }
return crc;
}
@@ -1067,10 +1097,12 @@ static void GetFileInfo(DebugFileInfo *dfi, const char *filename)
FILETIME write_time;
uint32 crc = (uint32)-1;
- file = CreateFile(filename, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, 0, 0);
+ file = CreateFile(filename, GENERIC_READ, FILE_SHARE_READ, NULL,
+ OPEN_EXISTING, 0, 0);
if (file != INVALID_HANDLE_VALUE) {
while(true) {
- if (ReadFile(file, buffer, sizeof(buffer), &numread, NULL) == 0 || numread==0)
+ if (ReadFile(file, buffer, sizeof(buffer), &numread, NULL) == 0 ||
+ numread == 0)
break;
filesize += numread;
crc = CalcCRC(buffer, numread, crc);
@@ -1091,6 +1123,7 @@ static char *PrintModuleInfo(char *output, HMODULE mod)
{
char buffer[MAX_PATH];
DebugFileInfo dfi;
+
GetModuleFileName(mod, buffer, MAX_PATH);
GetFileInfo(&dfi, buffer);
output += sprintf(output, " %-20s handle: %.8X size: %d crc: %.8X date: %d-%.2d-%.2d %.2d:%.2d:%.2d\r\n",
@@ -1123,8 +1156,9 @@ static char *PrintModuleList(char *output)
res = EnumProcessModules(proc, modules, sizeof(modules), &needed);
CloseHandle(proc);
if (res) {
- count = min(needed/sizeof(HMODULE*), sizeof(modules)/sizeof(HMODULE*));
- for(i=0; i!=count; i++)
+ count =
+ min(needed / sizeof(HMODULE), lengthof(modules));
+ for (i = 0; i != count; i++)
output = PrintModuleInfo(output, modules[i]);
return output;
}
@@ -1137,13 +1171,14 @@ static char *PrintModuleList(char *output)
static const char _crash_desc[] =
"A serious fault condition occured in the game. The game will shut down.\n"
"Press \"Submit report\" to send crash information to the developers. "
- "This will greatly help debugging. The information contained in the report is "
- "displayed below.\n"
+ "This will greatly help debugging. "
+ "The information contained in the report is displayed below.\n"
"Press \"Emergency save\" to attempt saving the game.";
static const char _save_succeeded[] =
- "Emergency save succeeded.\nBe aware that critical parts of the internal game state "
- "may have become corrupted. The saved game is not guaranteed to work.";
+ "Emergency save succeeded.\n"
+ "Be aware that critical parts of the internal game state may have become "
+ "corrupted. The saved game is not guaranteed to work.";
bool EmergencySave();
@@ -1255,12 +1290,13 @@ static void SetWndSize(HWND wnd, int mode)
if (mode >= 0) {
GetWindowRect(GetDlgItem(wnd, 11), &r2);
offs = r2.bottom - r2.top + 10;
- if (!mode) offs=-offs;
- SetWindowPos(wnd, HWND_TOPMOST, 0, 0, r.right - r.left, r.bottom - r.top + offs, SWP_NOMOVE | SWP_NOZORDER);
+ if (!mode) offs = -offs;
+ SetWindowPos(wnd, HWND_TOPMOST, 0, 0,
+ r.right - r.left, r.bottom - r.top + offs, SWP_NOMOVE | SWP_NOZORDER);
} else {
SetWindowPos(wnd, HWND_TOPMOST,
- (GetSystemMetrics(SM_CXSCREEN) - (r.right - r.left)) >> 1,
- (GetSystemMetrics(SM_CYSCREEN) - (r.bottom - r.top)) >> 1,
+ (GetSystemMetrics(SM_CXSCREEN) - (r.right - r.left)) / 2,
+ (GetSystemMetrics(SM_CYSCREEN) - (r.bottom - r.top)) / 2,
0, 0, SWP_NOSIZE);
}
}
@@ -1350,7 +1386,7 @@ static LONG WINAPI ExceptionHandler(EXCEPTION_POINTERS *ep)
char *output;
static bool had_exception;
- if (had_exception) { ExitProcess(0); }
+ if (had_exception) ExitProcess(0);
had_exception = true;
_ident = GetTickCount(); // something pretty unique
@@ -1375,7 +1411,8 @@ static LONG WINAPI ExceptionHandler(EXCEPTION_POINTERS *ep)
);
}
- if (_exception_string) output += sprintf(output, "Reason: %s\r\n", _exception_string);
+ if (_exception_string)
+ output += sprintf(output, "Reason: %s\r\n", _exception_string);
output += sprintf(output, "Exception %.8X at %.8X\r\n"
"Registers:\r\n"
@@ -1400,7 +1437,7 @@ static LONG WINAPI ExceptionHandler(EXCEPTION_POINTERS *ep)
{
byte *b = (byte*)ep->ContextRecord->Eip;
int i;
- for(i=0; i!=24; i++) {
+ for (i = 0; i != 24; i++) {
if (IsBadReadPtr(b, 1)) {
output += sprintf(output, " ??"); // OCR: WAS: , 0);
} else {
@@ -1417,8 +1454,8 @@ static LONG WINAPI ExceptionHandler(EXCEPTION_POINTERS *ep)
{
int i,j;
uint32 *b = (uint32*)ep->ContextRecord->Esp;
- for(j=0; j!=24; j++) {
- for(i=0; i!=8; i++) {
+ for (j = 0; j != 24; j++) {
+ for (i = 0; i != 8; i++) {
if (IsBadReadPtr(b,sizeof(uint32))) {
output += sprintf(output, " ????????"); //OCR: WAS - , 0);
} else {
@@ -1438,7 +1475,8 @@ static LONG WINAPI ExceptionHandler(EXCEPTION_POINTERS *ep)
os.dwOSVersionInfoSize = sizeof(os);
GetVersionEx(&os);
output += sprintf(output, "\r\nSystem information:\r\n"
- " Windows version %d.%d %d %s\r\n", os.dwMajorVersion, os.dwMinorVersion, os.dwBuildNumber, os.szCSDVersion);
+ " Windows version %d.%d %d %s\r\n",
+ os.dwMajorVersion, os.dwMinorVersion, os.dwBuildNumber, os.szCSDVersion);
}
{
@@ -1466,7 +1504,7 @@ static LONG WINAPI ExceptionHandler(EXCEPTION_POINTERS *ep)
static void Win32InitializeExceptions(void)
{
_asm {
- mov _safe_esp,esp
+ mov _safe_esp, esp
}
SetUnhandledExceptionFilter(ExceptionHandler);
@@ -1507,14 +1545,17 @@ static HANDLE MyFindFirstFile(const char *path, const char *file,
}
int CDECL compare_FiosItems (const void *a, const void *b) {
- const FiosItem *da = (const FiosItem *) a;
- const FiosItem *db = (const FiosItem *) b;
+ const FiosItem *da = (const FiosItem *)a;
+ const FiosItem *db = (const FiosItem *)b;
int r;
if (_savegame_sort_order < 2) // sort by date
r = da->mtime < db->mtime ? -1 : 1;
else
- r = stricmp(da->title[0] ? da->title : da->name, db->title[0] ? db->title : db->name);
+ r = stricmp(
+ da->title[0] != '\0' ? da->title : da->name,
+ db->title[0] != '\0' ? db->title : db->name
+ );
if (_savegame_sort_order & 1) r = -r;
return r;
@@ -1552,7 +1593,8 @@ FiosItem *FiosGetSavegameList(int *num, int mode)
if (h != INVALID_HANDLE_VALUE) {
do {
if (fd.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY &&
- !(fd.cFileName[0] == '.' && (fd.cFileName[1] == 0 || (fd.cFileName[1] == '.' && fd.cFileName[2] == 0)))) {
+ strcmp(fd.cFileName, ".") != 0 &&
+ strcmp(fd.cFileName, "..") != 0) {
fios = FiosAlloc();
fios->type = FIOS_TYPE_DIR;
strcpy(fios->name, fd.cFileName);
@@ -1585,7 +1627,11 @@ FiosItem *FiosGetSavegameList(int *num, int mode)
ttd_strlcpy(fios->name, fd.cFileName, strlen(fd.cFileName)-3);
} else if (mode == SLD_LOAD_GAME || mode == SLD_LOAD_SCENARIO) {
int ext = 0; // start of savegame extensions in _old_extensions[]
- if (t && ((ext++, !stricmp(t, ".SS1")) || (ext++, !stricmp(t, ".SV1")) || (ext++, !stricmp(t, ".SV2"))) ) { // TTDLX(Patch)
+ if (t != NULL && (
+ (ext++, stricmp(t, ".SS1") == 0) ||
+ (ext++, stricmp(t, ".SV1") == 0) ||
+ (ext++, stricmp(t, ".SV2") == 0)
+ )) { // TTDLX(Patch)
fios = FiosAlloc();
fios->old_extension = ext-1;
fios->mtime = *(uint64*)&fd.ftLastWriteTime;
@@ -1639,7 +1685,7 @@ FiosItem *FiosGetScenarioList(int *num, int mode)
_fios_path = _fios_scn_path;
// Parent directory, only if not of the type C:\.
- if (_fios_path[3] != 0 && mode != SLD_NEW_GAME) {
+ if (_fios_path[3] != '\0' && mode != SLD_NEW_GAME) {
fios = FiosAlloc();
fios->type = FIOS_TYPE_PARENT;
strcpy(fios->title, ".. (Parent directory)");
@@ -1650,7 +1696,8 @@ FiosItem *FiosGetScenarioList(int *num, int mode)
if (h != INVALID_HANDLE_VALUE && mode != SLD_NEW_GAME) {
do {
if (fd.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY &&
- !(fd.cFileName[0] == '.' && (fd.cFileName[1] == 0 || (fd.cFileName[1] == '.' && fd.cFileName[2] == 0)))) {
+ strcmp(fd.cFileName, ".") != 0 &&
+ strcmp(fd.cFileName, "..") != 0) {
fios = FiosAlloc();
fios->type = FIOS_TYPE_DIR;
strcpy(fios->name, fd.cFileName);
@@ -1673,16 +1720,22 @@ FiosItem *FiosGetScenarioList(int *num, int mode)
do {
if (!(fd.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY)) {
char *t = strrchr(fd.cFileName, '.');
- if (t && !stricmp(t, ".SCN")) { // OpenTTD
+
+ if (t != NULL && !stricmp(t, ".SCN")) { // OpenTTD
fios = FiosAlloc();
fios->mtime = *(uint64*)&fd.ftLastWriteTime;
sprintf(buf, "%s\\%s", _fios_path, fd.cFileName);
fios->type = FIOS_TYPE_SCENARIO;
- fios->title[0] = 0;
+ fios->title[0] = '\0';
ttd_strlcpy(fios->name, fd.cFileName, strlen(fd.cFileName)-3);
- } else if (mode == SLD_LOAD_GAME || mode == SLD_LOAD_SCENARIO || mode == SLD_NEW_GAME) {
+ } else if (mode == SLD_LOAD_GAME || mode == SLD_LOAD_SCENARIO ||
+ mode == SLD_NEW_GAME) {
int ext = 3; // start of scenario extensions in _old_extensions[]
- if (t && ((ext++, !stricmp(t, ".SV0")) || (ext++, !stricmp(t, ".SS0"))) ) { // TTDLX(Patch)
+
+ if (t != NULL && (
+ (ext++, stricmp(t, ".SV0") == 0) ||
+ (ext++, stricmp(t, ".SS0") == 0)
+ )) { // TTDLX(Patch)
fios = FiosAlloc();
fios->old_extension = ext-1;
fios->mtime = *(uint64*)&fd.ftLastWriteTime;
@@ -1702,16 +1755,14 @@ FiosItem *FiosGetScenarioList(int *num, int mode)
// Drives
if (mode != SLD_NEW_GAME) {
char drives[256];
- char *s;
+ const char *s;
+
GetLogicalDriveStrings(sizeof(drives), drives);
- s=drives;
- while (*s) {
+ for (s = drives; *s != '\0';) {
fios = FiosAlloc();
fios->type = FIOS_TYPE_DRIVE;
- fios->title[0] = s[0];
- fios->title[1] = ':';
- fios->title[2] = 0;
- while (*s++) {}
+ sprintf(fios->title, "%c:", s[0]);
+ while (*s++ != '\0') {}
}
}
@@ -1732,28 +1783,28 @@ char *FiosBrowseTo(const FiosItem *item)
{
static char str_buffr[512];
char *path = _fios_path;
- char *s;
- switch(item->type) {
+ switch (item->type) {
case FIOS_TYPE_DRIVE:
sprintf(path, "%c:\\", item->title[0]);
break;
- case FIOS_TYPE_PARENT:
+ case FIOS_TYPE_PARENT: {
+ char *s;
+
// Skip drive part
path += 3;
s = path;
- while (*path) {
- if (*path== '\\')
- s = path;
- path++;
+ for (; *path != '\0'; path++) {
+ if (*path== '\\') s = path;
}
- *s = 0;
+ *s = '\0';
break;
+ }
case FIOS_TYPE_DIR:
// Scan to end
- while (*++path);
+ while (*++path != '\0');
// Add backslash?
if (path[-1] != '\\') *path++ = '\\';
@@ -1765,14 +1816,17 @@ char *FiosBrowseTo(const FiosItem *item)
return str_buffr;
case FIOS_TYPE_OLDFILE:
- sprintf(str_buffr, "%s\\%s.%s", _fios_path, item->name, _old_extensions[item->old_extension]);
+ sprintf(str_buffr, "%s\\%s.%s",
+ _fios_path, item->name, _old_extensions[item->old_extension]);
return str_buffr;
case FIOS_TYPE_SCENARIO:
sprintf(str_buffr, "%s\\%s.scn", path, item->name);
return str_buffr;
+
case FIOS_TYPE_OLD_SCENARIO:
- sprintf(str_buffr, "%s\\%s.%s", path, item->name, _old_extensions[item->old_extension]);
+ sprintf(str_buffr, "%s\\%s.%s",
+ path, item->name, _old_extensions[item->old_extension]);
return str_buffr;
}
@@ -1788,12 +1842,9 @@ StringID FiosGetDescText(const char **path)
DWORD spc, bps, nfc, tnc;
*path = _fios_path;
- root[0] = _fios_path[0];
- root[1] = ':';
- root[2] = '\\';
- root[3] = 0;
+ sprintf(root, "%c:\\", _fios_path[0]);
if (GetDiskFreeSpace(root, &spc, &bps, &nfc, &tnc)) {
- uint32 tot = ((spc*bps)*(uint64)nfc) >> 20;
+ uint32 tot = ((spc * bps) * (uint64)nfc) >> 20;
SetDParam(0, tot);
return STR_4005_BYTES_FREE;
} else {
@@ -1803,7 +1854,7 @@ StringID FiosGetDescText(const char **path)
void FiosMakeSavegameName(char *buf, const char *name)
{
- if(_game_mode == GM_EDITOR)
+ if (_game_mode == GM_EDITOR)
sprintf(buf, "%s\\%s.scn", _fios_path, name);
else
sprintf(buf, "%s\\%s.sav", _fios_path, name);
@@ -1928,24 +1979,24 @@ static int ParseCommandLine(char *line, char **argv, int max_argc)
line++;
// end?
- if (*line == 0)
+ if (*line == '\0')
break;
// special handling when quoted
if (*line == '"') {
argv[n++] = ++line;
while (*line != '"') {
- if (*line == 0) return n;
+ if (*line == '\0') return n;
line++;
}
} else {
argv[n++] = line;
while (*line != ' ' && *line != '\t') {
- if (*line == 0) return n;
+ if (*line == '\0') return n;
line++;
}
}
- *line++ = 0;
+ *line++ = '\0';
} while (n != max_argc);
return n;
@@ -1953,7 +2004,7 @@ static int ParseCommandLine(char *line, char **argv, int max_argc)
#if defined(_MSC_VER)
-__int64 _declspec(naked) rdtsc()
+static uint64 _declspec(naked) rdtsc(void)
{
_asm {
rdtsc
@@ -1990,9 +2041,9 @@ void CreateConsole(void)
*stderr = *fdopen(2, "w" );
#endif
- setvbuf( stdin, NULL, _IONBF, 0 );
- setvbuf( stdout, NULL, _IONBF, 0 );
- setvbuf( stderr, NULL, _IONBF, 0 );
+ setvbuf(stdin, NULL, _IONBF, 0);
+ setvbuf(stdout, NULL, _IONBF, 0);
+ setvbuf(stderr, NULL, _IONBF, 0);
}
void ShowInfo(const char *str)
@@ -2014,7 +2065,8 @@ void ShowInfo(const char *str)
}
-int APIENTRY WinMain(HINSTANCE hInstance,HINSTANCE hPrevInstance,LPTSTR lpCmdLine,int nCmdShow)
+int APIENTRY WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance,
+ LPTSTR lpCmdLine, int nCmdShow)
{
int argc;
char *argv[64]; // max 64 command line arguments
@@ -2031,8 +2083,8 @@ int APIENTRY WinMain(HINSTANCE hInstance,HINSTANCE hPrevInstance,LPTSTR lpCmdLin
#if defined(_MSC_VER)
{
uint64 seed = rdtsc();
- _random_seeds[0][0] = ((uint32*)&seed)[0];
- _random_seeds[0][1] = ((uint32*)&seed)[1];
+ _random_seeds[0][0] = seed & 0xffffffff;
+ _random_seeds[0][1] = seed >> 32;
}
#else
_random_seeds[0][0] = GetTickCount();
@@ -2070,7 +2122,7 @@ void DeterminePaths(void)
s = strchr(cfg, 0);
- if (s[-1] != '\\') { s[0] = '\\'; s[1] = 0; }
+ if (s[-1] != '\\') strcpy(s, "\\");
_path.save_dir = str_fmt("%ssave", cfg);
_path.autosave_dir = str_fmt("%s\\autosave", _path.save_dir);