Mercurial > sdl-ios-xcode
annotate src/thread/win32/SDL_systhread.c @ 1257:448a9a64537b
[PATCH] SDL_GetVideoMode() does not find best mode, part 2
Following commit 1.51, I come accross a problem when SDL must choose between
several video modes that could suit the one asked.
If I ask 320x240 with this list:
768x480 768x240 640x400 640x200 384x480 384x240 320x400 320x200
The smallest selectables modes are 384x240 and 320x400. And SDL choose the later
in this list, but 384x240 is more suitable. So I added a check to compare
the pixel count (surface) of modes, and select the one which has the smallest
pixel count.
In my example, 384x240 has 92160 pixels, and 320x400 has 128000 pixels. So now
SDL will choose 384x240 for the asked 320x240 mode.
| author | Patrice Mandin <patmandin@gmail.com> |
|---|---|
| date | Thu, 19 Jan 2006 21:28:52 +0000 |
| parents | 86d0d01290ea |
| children | c9b51268668f |
| rev | line source |
|---|---|
| 0 | 1 /* |
| 2 SDL - Simple DirectMedia Layer | |
|
769
b8d311d90021
Updated copyright information for 2004 (Happy New Year!)
Sam Lantinga <slouken@libsdl.org>
parents:
297
diff
changeset
|
3 Copyright (C) 1997-2004 Sam Lantinga |
| 0 | 4 |
| 5 This library is free software; you can redistribute it and/or | |
| 6 modify it under the terms of the GNU Library General Public | |
| 7 License as published by the Free Software Foundation; either | |
| 8 version 2 of the License, or (at your option) any later version. | |
| 9 | |
| 10 This library is distributed in the hope that it will be useful, | |
| 11 but WITHOUT ANY WARRANTY; without even the implied warranty of | |
| 12 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | |
| 13 Library General Public License for more details. | |
| 14 | |
| 15 You should have received a copy of the GNU Library General Public | |
| 16 License along with this library; if not, write to the Free | |
| 17 Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA | |
| 18 | |
| 19 Sam Lantinga | |
|
252
e8157fcb3114
Updated the source with the correct e-mail address
Sam Lantinga <slouken@libsdl.org>
parents:
0
diff
changeset
|
20 slouken@libsdl.org |
| 0 | 21 */ |
| 22 | |
| 23 #ifdef SAVE_RCSID | |
| 24 static char rcsid = | |
| 25 "@(#) $Id$"; | |
| 26 #endif | |
| 27 | |
| 28 /* Win32 thread management routines for SDL */ | |
| 29 | |
| 30 #include <stdio.h> | |
| 31 #include <stdlib.h> | |
| 32 #include <windows.h> | |
|
1251
86d0d01290ea
Updated Windows CE/PocketPC support...adds GAPI driver, landscape mode,
Ryan C. Gordon <icculus@icculus.org>
parents:
1225
diff
changeset
|
33 |
|
86d0d01290ea
Updated Windows CE/PocketPC support...adds GAPI driver, landscape mode,
Ryan C. Gordon <icculus@icculus.org>
parents:
1225
diff
changeset
|
34 #ifndef _WIN32_WCE |
|
1225
09bc22169702
Windows should use _beginthreadex() instead of CreateThread(), to avoid a
Ryan C. Gordon <icculus@icculus.org>
parents:
769
diff
changeset
|
35 #include <process.h> |
|
1251
86d0d01290ea
Updated Windows CE/PocketPC support...adds GAPI driver, landscape mode,
Ryan C. Gordon <icculus@icculus.org>
parents:
1225
diff
changeset
|
36 #endif |
| 0 | 37 |
| 38 #include "SDL_error.h" | |
| 39 #include "SDL_thread.h" | |
| 40 #include "SDL_systhread.h" | |
| 41 | |
| 42 | |
|
1225
09bc22169702
Windows should use _beginthreadex() instead of CreateThread(), to avoid a
Ryan C. Gordon <icculus@icculus.org>
parents:
769
diff
changeset
|
43 static unsigned __stdcall RunThread(void *data) |
| 0 | 44 { |
| 45 SDL_RunThread(data); | |
| 46 return(0); | |
| 47 } | |
| 48 | |
| 49 int SDL_SYS_CreateThread(SDL_Thread *thread, void *args) | |
| 50 { | |
|
1225
09bc22169702
Windows should use _beginthreadex() instead of CreateThread(), to avoid a
Ryan C. Gordon <icculus@icculus.org>
parents:
769
diff
changeset
|
51 unsigned threadid; |
| 0 | 52 |
|
1225
09bc22169702
Windows should use _beginthreadex() instead of CreateThread(), to avoid a
Ryan C. Gordon <icculus@icculus.org>
parents:
769
diff
changeset
|
53 /* |
|
09bc22169702
Windows should use _beginthreadex() instead of CreateThread(), to avoid a
Ryan C. Gordon <icculus@icculus.org>
parents:
769
diff
changeset
|
54 * Avoid CreateThread: https://bugzilla.libsdl.org/show_bug.cgi?id=22 |
|
09bc22169702
Windows should use _beginthreadex() instead of CreateThread(), to avoid a
Ryan C. Gordon <icculus@icculus.org>
parents:
769
diff
changeset
|
55 * |
|
09bc22169702
Windows should use _beginthreadex() instead of CreateThread(), to avoid a
Ryan C. Gordon <icculus@icculus.org>
parents:
769
diff
changeset
|
56 * have to use _beginthreadex if we want the returned handle |
|
09bc22169702
Windows should use _beginthreadex() instead of CreateThread(), to avoid a
Ryan C. Gordon <icculus@icculus.org>
parents:
769
diff
changeset
|
57 * to be accessible after the thread exits |
|
09bc22169702
Windows should use _beginthreadex() instead of CreateThread(), to avoid a
Ryan C. Gordon <icculus@icculus.org>
parents:
769
diff
changeset
|
58 * threads created with _beginthread auto-close the handle |
|
1251
86d0d01290ea
Updated Windows CE/PocketPC support...adds GAPI driver, landscape mode,
Ryan C. Gordon <icculus@icculus.org>
parents:
1225
diff
changeset
|
59 * Windows CE still use CreateThread. |
|
1225
09bc22169702
Windows should use _beginthreadex() instead of CreateThread(), to avoid a
Ryan C. Gordon <icculus@icculus.org>
parents:
769
diff
changeset
|
60 */ |
|
1251
86d0d01290ea
Updated Windows CE/PocketPC support...adds GAPI driver, landscape mode,
Ryan C. Gordon <icculus@icculus.org>
parents:
1225
diff
changeset
|
61 #ifdef _WIN32_WCE |
|
86d0d01290ea
Updated Windows CE/PocketPC support...adds GAPI driver, landscape mode,
Ryan C. Gordon <icculus@icculus.org>
parents:
1225
diff
changeset
|
62 thread->handle = CreateThread(NULL, 0, RunThread, args, 0, &threadid); |
|
86d0d01290ea
Updated Windows CE/PocketPC support...adds GAPI driver, landscape mode,
Ryan C. Gordon <icculus@icculus.org>
parents:
1225
diff
changeset
|
63 #else |
|
1225
09bc22169702
Windows should use _beginthreadex() instead of CreateThread(), to avoid a
Ryan C. Gordon <icculus@icculus.org>
parents:
769
diff
changeset
|
64 thread->handle = (SYS_ThreadHandle) _beginthreadex(NULL, 0, RunThread, |
|
09bc22169702
Windows should use _beginthreadex() instead of CreateThread(), to avoid a
Ryan C. Gordon <icculus@icculus.org>
parents:
769
diff
changeset
|
65 args, 0, &threadid); |
|
1251
86d0d01290ea
Updated Windows CE/PocketPC support...adds GAPI driver, landscape mode,
Ryan C. Gordon <icculus@icculus.org>
parents:
1225
diff
changeset
|
66 #endif |
| 0 | 67 if (thread->handle == NULL) { |
| 68 SDL_SetError("Not enough resources to create thread"); | |
| 69 return(-1); | |
| 70 } | |
| 71 return(0); | |
| 72 } | |
| 73 | |
| 74 void SDL_SYS_SetupThread(void) | |
| 75 { | |
| 76 return; | |
| 77 } | |
| 78 | |
| 79 Uint32 SDL_ThreadID(void) | |
| 80 { | |
| 81 return((Uint32)GetCurrentThreadId()); | |
| 82 } | |
| 83 | |
| 84 void SDL_SYS_WaitThread(SDL_Thread *thread) | |
| 85 { | |
| 86 WaitForSingleObject(thread->handle, INFINITE); | |
| 87 CloseHandle(thread->handle); | |
| 88 } | |
| 89 | |
| 90 /* WARNING: This function is really a last resort. | |
| 91 * Threads should be signaled and then exit by themselves. | |
| 92 * TerminateThread() doesn't perform stack and DLL cleanup. | |
| 93 */ | |
| 94 void SDL_SYS_KillThread(SDL_Thread *thread) | |
| 95 { | |
| 96 TerminateThread(thread->handle, FALSE); | |
| 97 } |