Re: [FFmpeg-devel] [PATCH 12/21] avcodec/smacker: Remove redundant check when decoding header trees

2020-08-01 Thread Paul B Mahol
probably fine On 8/1/20, Andreas Rheinhardt wrote: > 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

[FFmpeg-devel] [PATCH 12/21] avcodec/smacker: Remove redundant check when decoding header trees

2020-08-01 Thread Andreas Rheinhardt
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