[FFmpeg-devel] [PATCH v2] libavcodec/hevc_filter: support for all skip_loop_filter levels

Michael Niedermayer michael at niedermayer.cc
Mon Dec 4 19:49:41 EET 2017


On Sun, Dec 03, 2017 at 07:58:18PM +0000, Stefan _ wrote:
> On 01.12.2017 at 17:45 Michael Niedermayer wrote:
> > AVDISCARD_NONREF is about frames which are not referenced by any other
> > doesnt ff_hevc_frame_nb_refs() produce the number of references curently
> > aka the other end of the reference arrows ?
> 
> Yes, that sounds right.
> 
> I've revised the patch to check for *_N NAL unit types instead (which 
> should be equivalent to H.264's nal_idc == 0) and to skip not only 
> deblock, but also SAO.
> 

>  doc/decoders.texi        |    7 -------
>  libavcodec/hevc_filter.c |   29 +++++++++++++++++++++++++++--
>  2 files changed, 27 insertions(+), 9 deletions(-)
> d0c4c6a91d7222fd9f370bd213e1075d7551463b  0001-libavcodec-hevc_filter-support-for-all-skip_loop_fil.patch
> From a3ebeb4ca0849cfc3846de79b38ae5ca6c001718 Mon Sep 17 00:00:00 2001
> From: sfan5 <sfan5 at live.de>
> Date: Thu, 30 Nov 2017 23:58:02 +0100
> Subject: [PATCH] libavcodec/hevc_filter: support for all skip_loop_filter
>  levels.
> 
> Continues where commit 52c75d486ed5f75cbb79e5dbd07b7aef24f3071f left off.
> ---
>  doc/decoders.texi        |  7 -------
>  libavcodec/hevc_filter.c | 29 +++++++++++++++++++++++++++--
>  2 files changed, 27 insertions(+), 9 deletions(-)

will apply

can you add a fate test for this ?

thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Rewriting code that is poorly written but fully understood is good.
Rewriting code that one doesnt understand is a sign that one is less smart
then the original author, trying to rewrite it will not make it better.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20171204/c6289de0/attachment.sig>


More information about the ffmpeg-devel mailing list