[FFmpeg-devel] [PATCH 6/7] avcodec/ffv1: support of more pix_fmt

Michael Niedermayer michael at niedermayer.cc
Sat Mar 10 17:44:13 EET 2018


On Fri, Mar 09, 2018 at 08:56:56PM +0100, Jerome Martinez wrote:
> On 09/03/2018 18:30, Paul B Mahol wrote:
> >On 3/7/18, Jerome Martinez <jerome at mediaarea.net> wrote:
> >>With some sources having specific pix_fmt (9/10/12/14 bit), the source
> >>is padded to 16-bit when the pix_fmt is not supported natively by the
> >>FFV1 encoder.
> >>Nothing is lost ("cutting" to the source bitdepth permits to retrieve
> >>the exact source), but the source bitdepth is not indicated so it is not
> >>possible to retrieve the exact source without having the source bitdepth
> >>by another channel. It also makes FATE tests (framemd5 comparison) a bit
> >>more complicated (bitdepth reduction).
> >>
> >>This patch adds native support of the pix_fmt being padded, so there is
> >>no more padding and the FFV1 bitdepth is same as source bitdepth.
> >>
> >Trailing whitespaces are not allowed in FFmpeg codebase.
> 
> Was not intended.
> Updated patch attached.

>  ffv1dec.c |   14 +++++++++++++-
>  ffv1enc.c |   13 ++++++++++++-
>  2 files changed, 25 insertions(+), 2 deletions(-)
> 5f129d841fc5f96067f48697c156a04b3cacf33c  0001-avcodec-ffv1-support-of-more-pix_fmt.patch
> From 6e9a50ba90c3264c26b4e775e0618ee1702812a8 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?J=C3=A9r=C3=B4me=20Martinez?= <jerome at mediaarea.net>
> Date: Wed, 7 Mar 2018 11:19:03 +0100
> Subject: [PATCH] avcodec/ffv1: support of more pix_fmt
> 
> Without direct support of such pix_fmt, content is padded to 16-bit
> and it is not possible to know that the source file was with a smaller bit depth
> so framemd5 is different
> ---
>  libavcodec/ffv1dec.c | 14 +++++++++++++-
>  libavcodec/ffv1enc.c | 13 ++++++++++++-
>  2 files changed, 25 insertions(+), 2 deletions(-)

will apply
thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

While the State exists there can be no freedom; when there is freedom there
will be no State. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180310/423c6a73/attachment.sig>


More information about the ffmpeg-devel mailing list