test/testtypes.c
author Patrice Mandin <patmandin@gmail.com>
Thu, 19 Jan 2006 21:28:52 +0000
changeset 1257 448a9a64537b
parent 1154 d93862a3d821
child 1387 835c1831f903
permissions -rw-r--r--
[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.
slouken@0
     1
slouken@0
     2
#include <stdio.h>
icculus@1154
     3
#include <stdlib.h>
icculus@1154
     4
#include <string.h>
slouken@47
     5
#include "SDL_main.h"
slouken@0
     6
#include "SDL_types.h"
slouken@0
     7
icculus@1154
     8
/*
icculus@1154
     9
 * Watcom C flags these as Warning 201: "Unreachable code" if you just
icculus@1154
    10
 *  compare them directly, so we push it through a function to keep the
icculus@1154
    11
 *  compiler quiet.  --ryan.
icculus@1154
    12
 */
icculus@1154
    13
static int badsize(size_t sizeoftype, size_t hardcodetype)
icculus@1154
    14
{
icculus@1154
    15
    return sizeoftype != hardcodetype;
icculus@1154
    16
}
icculus@1154
    17
slouken@0
    18
int main(int argc, char *argv[])
slouken@0
    19
{
slouken@0
    20
	int error = 0;
slouken@0
    21
	int verbose = 1;
slouken@0
    22
slouken@0
    23
	if ( argv[1] && (strcmp(argv[1], "-q") == 0) )
slouken@0
    24
		verbose = 0;
slouken@0
    25
icculus@1154
    26
	if ( badsize(sizeof(Uint8), 1) ) {
slouken@0
    27
		if ( verbose )
slouken@0
    28
			printf("sizeof(Uint8) != 1, instead = %d\n",
slouken@0
    29
								sizeof(Uint8));
slouken@0
    30
		++error;
slouken@0
    31
	}
icculus@1154
    32
	if ( badsize(sizeof(Uint16), 2) ) {
slouken@0
    33
		if ( verbose )
slouken@0
    34
			printf("sizeof(Uint16) != 2, instead = %d\n",
slouken@0
    35
								sizeof(Uint16));
slouken@0
    36
		++error;
slouken@0
    37
	}
icculus@1154
    38
	if ( badsize(sizeof(Uint32), 4) ) {
slouken@0
    39
		if ( verbose )
slouken@0
    40
			printf("sizeof(Uint32) != 4, instead = %d\n",
slouken@0
    41
								sizeof(Uint32));
slouken@0
    42
		++error;
slouken@0
    43
	}
slouken@0
    44
#ifdef SDL_HAS_64BIT_TYPE
icculus@1154
    45
	if ( badsize(sizeof(Uint64), 8) ) {
slouken@0
    46
		if ( verbose )
slouken@0
    47
			printf("sizeof(Uint64) != 8, instead = %d\n",
slouken@0
    48
								sizeof(Uint64));
slouken@0
    49
		++error;
slouken@0
    50
	}
slouken@0
    51
#else
slouken@0
    52
	if ( verbose ) {
slouken@0
    53
		printf("WARNING: No 64-bit datatype on this platform\n");
slouken@0
    54
	}
slouken@0
    55
#endif
slouken@0
    56
	if ( verbose && ! error )
slouken@0
    57
		printf("All data types are the expected size.\n");
slouken@0
    58
slouken@0
    59
	return( error ? 1 : 0 );
slouken@0
    60
}