[MPlayer-dev-eng] [PATCH] vo_tga: fix blue channel on little endian machines
Marton Balint
cus at fazekas.hu
Sun Jul 11 22:13:10 CEST 2010
On Sat, 26 Jun 2010, Marton Balint wrote:
> On Sat, 26 Jun 2010, Reimar Döffinger wrote:
>> On Sat, Jun 26, 2010 at 11:08:24AM +0200, Marton Balint wrote:
>>> @@ -236,7 +196,7 @@
>>> switch(format){
>>> case IMGFMT_BGR|15:
>>> case IMGFMT_BGR|24:
>>> - case IMGFMT_BGR|32:
>>> + case IMGFMT_BGRA:
>>
>> Well, if your previous patch was correct, this is wrong, it should stay
>> IMGFMT_BGR|32.
>
> I think my previous patch was wrong, at the time of writing it I wasn't aware
> of the fact that IMGFMT_BGRA is endian dependent. Because of so I think using
> IMGFMT_BGRA is the correct approach. Am I missing something again? :)
Could someone please review the patch?
Regards,
Marton
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vo_tga-fix-blue-channel-3.patch
Type: text/x-patch
Size: 2624 bytes
Desc:
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20100711/33179504/attachment.bin>
More information about the MPlayer-dev-eng
mailing list