[FFmpeg-devel] [PATCH]lavc/libvpx: Fix high-bitrate colour spaces

James Zern jzern at google.com
Fri Feb 19 19:37:33 CET 2016


On Fri, Feb 19, 2016 at 12:43 AM, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
> Hendrik Leppkes <h.leppkes <at> gmail.com> writes:
>
>> On Fri, Feb 19, 2016 at 8:47 AM, James Zern wrote:
>> > On Thu, Feb 18, 2016 at 11:40 PM, Carl Eugen Hoyos wrote:
>> >> On Friday 19 February 2016 07:59:37 am James Zern wrote:
>> >>
>> >>> The decoder won't change when built on a big-endian
>> >>> platform, so I don't think this is correct.
>> >>
>> >> New patch attached.
>> >
>> > Sorry, I meant all of it, high bitdepth is no different.
>>
>> I think his point is that the decoder always outputs LE,
>> and this change is therefor incorrect?
>
> Don't you agree that this is highly unlikely?
> (I mean by looking at the code, not because I claim it.)
>

Yes, you're correct, I wasn't considering the high-bitdepth output
properly. The decoder uses 16-bit values in their native endian
format, so your patch is correct. Sorry for the churn.


More information about the ffmpeg-devel mailing list