DSPF_RBG15 in src/video/directfb/SDL_DirectFB_video.c

SDL Development
1

Hello all,

Just wanted to inform you that I had to change the three occurences of
DSPF_RGB15 in src/video/directfb/SDL_DirectFB_video.c to DSPF_ARGB1555
in order to get the CVS version of SDL to compile with the CVS version
of DirectFB… I’m not sure what the DirectFB support is like under SDL
but that got it to compile, at least.–
William <@William_Morgan>

2

Hi,

Just wanted to inform you that I had to change the three occurences of
DSPF_RGB15 in src/video/directfb/SDL_DirectFB_video.c to DSPF_ARGB1555
in order to get the CVS version of SDL to compile with the CVS version
of DirectFB… I’m not sure what the DirectFB support is like under SDL
but that got it to compile, at least.

I just checked the header files of DirectFB myself.
In my local DirectFB 0.1.19 copy, DSPF_RGB15 is an alias for DSPF_ARGB1555.
They seem to have it removed in DirectFB CVS.

As DSPF_RGB15 was only an alias, the attached patch
can safely be applied to SDL and won’t brake older DirectFB versions.

Cheers,
Thomas
-------------- next part --------------
A non-text attachment was scrubbed…
Name: SDL-directfb-ARGB1555.patch
Type: text/x-diff
Size: 944 bytes
Desc: not available
URL: http://lists.libsdl.org/pipermail/sdl-libsdl.org/attachments/20040128/a130c76d/attachment.patch

3

As DSPF_RGB15 was only an alias, the attached patch
can safely be applied to SDL and won’t brake older DirectFB versions.

Added to CVS.

Thanks!
-Sam Lantinga, Software Engineer, Blizzard Entertainment