diff mbox series

[FFmpeg-devel,5/8] avcodec/magicyuv: Don't zero unnecessarily

Message ID 20200831210930.18900-5-andreas.rheinhardt@gmail.com
State Accepted
Commit 7f452c099a47eb54131f4a36e6078471e24c39b4
Headers show
Series [FFmpeg-devel,1/8] avcodec/magicyuv: Don't invert Huffman table symbols twice | expand

Checks

Context Check Description
andriy/default pending
andriy/make success Make finished
andriy/make_fate success Make fate finished

Commit Message

Andreas Rheinhardt Aug. 31, 2020, 9:09 p.m. UTC 
The code already checks that exactly the expected amount of entries are
read and set. Ergo it is unnecessary to zero them at the beginning.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
---
 libavcodec/magicyuv.c | 1 -
 1 file changed, 1 deletion(-)
diff mbox series

Patch

diff --git a/libavcodec/magicyuv.c b/libavcodec/magicyuv.c
index f2204f3401..9282b39723 100644
--- a/libavcodec/magicyuv.c
+++ b/libavcodec/magicyuv.c
@@ -453,7 +453,6 @@  static int build_huffman(AVCodecContext *avctx, GetBitContext *gbit, int max)
     MagicYUVContext *s = avctx->priv_data;
     int i = 0, j = 0, k;
 
-    memset(s->len, 0, sizeof(s->len));
     while (get_bits_left(gbit) >= 8) {
         int b = get_bits(gbit, 1);
         int x = get_bits(gbit, 7);