Re: [sdljava-users] Re: Small patch to SDLVideo.mapRGBA

[Ivan Z. G. <iva...@ya...>]

No worries.  I see to have applied it correctly anyhow.  It was so small 
I typed it in.  I have also applied to fix to SDLSurface at this time.  
CVS on sourceforge is slugish right now but it will be checked in next 
time I can access the repository.

Thanks for finding these!

-Ivan/

Flavio Chierichetti wrote:

>On Mon, 2005-01-24 at 10:42 +0100, Flavio Chierichetti wrote:
>  
>
>>Hi,
>>	the mapRGBA method switch the green and blue component (the mapRGB
>>method works well). While the right order should be RGBA, the native
>>method is called with RBGA.
>>
>>This little patch fixes the problem (I took the file from cvs on the
>>21st of this month):
>>
>>457c457
>><       return SWIG_SDLVideo.SDL_MapRGBA(format.getSwigPixelFormat(),
>>(short)r, (short)g, (short)b, (short)a);
>>---
>>    
>>
>>>      return SWIG_SDLVideo.SDL_MapRGBA(format.getSwigPixelFormat(),
>>>      
>>>
>>(short)r, (short)b, (short)g, (short)a);
>>    
>>
>
>The same problem can be found in SDLSurface.java :
>
>939c939
><       return SDLVideo.mapRGBA(getFormat(), r, b, g, a);
>---
>  
>
>>      return SDLVideo.mapRGBA(getFormat(), r, g, b, a);
>>    
>>
>
>
>Ciao!
>   Flavio
>
>P.S.: I've made the previous patch the wrong way around *blush*. I hope
>this one is good ;)
>
>  
>