[FFmpeg-devel] [PATCH 1/2] (was Re: deduplicated [PATCH] Cinepak: speed up decoding several-fold...) formats.

Paul B Mahol onemda at gmail.com
Sun Mar 5 20:32:08 EET 2017


On 3/5/17, u-9iep at aetey.se <u-9iep at aetey.se> wrote:
> For one week there was no substantial feedback on the patches.
>
> I understand that the first patch was very large which makes it hard
> to review.
>
> That's why I now have produced a limited, minimalistic change which still
> yields most of the improvement.
>
> I kindly ask the reader to note that cinepak is not ffmpeg's everyday meal
> i.e. fast shortcut judgements may be not applicable. Please take your time.
>
> To avoid contention around some of the issues which triggered such
> regrettable kind of judgements let me point out that
>
> - letting a decoder produce multiple pixel formats is explicitly
>   allowed and supported by the documented API (get_format())
>
> - the patch does not introduce a new feature in this respect,
>   just makes its presence more visible and much more useful
>
> - the cinepak codec is extraordinary well suited to produce virtually
>   any desired pixel format very efficiently (iow it is quite special)

Cinepak is old crappy codec. Get over it.


More information about the ffmpeg-devel mailing list