[FFmpeg-devel] [PATCH] lavc/libvpx: fix support for RGB colorspace.

James Zern jzern at google.com
Fri Feb 19 08:16:14 CET 2016


On Thu, Feb 18, 2016 at 3:02 PM, Hendrik Leppkes <h.leppkes at gmail.com> wrote:
> On Thu, Feb 18, 2016 at 11:59 PM, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
>> On Thursday 18 February 2016 09:40:01 pm Nicolas George wrote:
>>> Initial patch by Carl Eugen Hoyos.
>>>
>>> Fix trac ticket #5249.
>>
>> Patch with (automatic) support for 8, 10 and 12 bit gbr attached.
>>
>> Please comment, Carl Eugen
>
> This lacks the appropriate changes in libvpx.c to the supported pixel
> formats for the encoder.
> Also, please re-indent appropriately. If its not in this patch, it'll
> never get done.
>

There was a historical preference not to do that, the developer docs
still say something to that effect [1]. If the preference has changed
then I'm OK with it.

[1] http://ffmpeg.org/developer.html#Development-Policy
"8. We refuse source indentation and other cosmetic changes if they
are mixed with functional changes, such commits will be rejected and
removed."


More information about the ffmpeg-devel mailing list