]> git.sesse.net Git - ffmpeg/commit
avcodec/smacker: Remove redundant check when decoding header trees
authorAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Fri, 24 Jul 2020 18:56:46 +0000 (20:56 +0200)
committerAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Thu, 17 Sep 2020 23:57:09 +0000 (01:57 +0200)
commit943a4581570728b4b1a2bf97e5601e24f3f80404
treea5789d5870876975c7104362e1fa2d82e8ffeb25
parent77c9507253cab75474825be0836dd147dd2ab602
avcodec/smacker: Remove redundant check when decoding header trees

When length is zero for a leaf node (which happens iff the Huffman tree
consists of one leaf node only), prefix is also automatically zero.

Performance impact is negligible: For GCC 9 and the sample from #2425,
the time for one call to smka_decode_frame() decreased from 2053758 to
2053671 decicycles; for Clang 9 it went from 1523153 to 1521288.

Reviewed-by: Paul B Mahol <onemda@gmail.com>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
libavcodec/smacker.c