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 <jer...@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?= <jer...@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

Attachment: signature.asc
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to