[FFmpeg-devel] [PATCH 1/2] avcodec/fitsdec: change to be pixel formats

Carl Eugen Hoyos ceffmpeg at gmail.com
Sat Sep 16 00:43:41 EEST 2017


2017-09-15 23:37 GMT+02:00 James Almer <jamrial at gmail.com>:
> On 9/15/2017 6:34 PM, Carl Eugen Hoyos wrote:
>> 2017-09-15 22:30 GMT+02:00 Paras Chadha <paraschadha18 at gmail.com>:
>>
>>> -                    *dst++ = (type) t; \
>>> +                    write(dst, (type) t); \
>>
>> Is the cast still necessary?
>>
>> Is the new code faster on any platform?
>
> It fixes the fate tests on big endian systems, which is the
> purpose of this change.

I just wonder if the original patch wasn't cleaner.

Carl Eugen


More information about the ffmpeg-devel mailing list