[MPlayer-dev-eng] [PATCH] vo_tga: fix blue channel on little endian machines

Marton Balint cus at fazekas.hu
Sat Jun 26 11:08:24 CEST 2010


On Thu, 10 Jun 2010, Marton Balint wrote:
>
> On Wed, 9 Jun 2010, Reimar Döffinger wrote:
>> On Wed, Jun 09, 2010 at 09:16:57PM +0200, Marton Balint wrote:
>>> +            while (dy-- > 0) {
>>> +                if (fwrite(buf, wb, 1, fo) != 1) {
>>> +                    er = 4;
>>> +                    break;
>>>                  }
>>> -
>>> +                buf += stride;
>>>              }
>>> -            else {
>>> -                while (dy-- > 0) {
>>> -                    if (fwrite(buf, wb, 1, fo) != 1) {
>>> -                        er = 4;
>>> -                        break;
>>> -                    }
>>> -                    buf += stride;
>> 
>> Please don't change indentation, it's easier to check for mistakes without 
>> it
>> and then fix the indentation later.
>
> Sorry, I fixed the patch to use the original identation.
>
>>> @@ -236,7 +196,11 @@
>>>      switch(format){
>>>          case IMGFMT_BGR|15:
>>>          case IMGFMT_BGR|24:
>>> -        case IMGFMT_BGR|32:
>>> +        #if HAVE_BIGENDIAN
>>> +        case IMGFMT_ARGB:
>>> +        #else
>>> +        case IMGFMT_BGRA:
>>> +        #endif
>> 
>> That's just a more complicated way of writing the same thing actually,
>> so this part should be unnecessary.
>
> OK, I removed the HAVE_BIGENDIAN check.

Any news on the patch? Is it acceptable in it's current form?

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/20100626/d992a484/attachment.bin>


More information about the MPlayer-dev-eng mailing list