From: Peter Ross <pross@xvid.org> To: ffmpeg-devel@ffmpeg.org Subject: [FFmpeg-devel] [PATCH] avcodec/rv60dec: drop unused sum variable in read_slice_sizes Date: Sun, 22 Jun 2025 14:33:10 +1000 Message-ID: <4cf47707d29878f27286220f42481f8651f8ed50.1750566661.git.pross@xvid.org> (raw) [-- Attachment #1.1: Type: text/plain, Size: 1214 bytes --] --- libavcodec/rv60dec.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/libavcodec/rv60dec.c b/libavcodec/rv60dec.c index 2bbcb1d620..6075598861 100644 --- a/libavcodec/rv60dec.c +++ b/libavcodec/rv60dec.c @@ -390,14 +390,14 @@ static int read_frame_header(RV60Context *s, GetBitContext *gb, int * width, int static int read_slice_sizes(RV60Context *s, GetBitContext *gb) { int nbits = get_bits(gb, 5) + 1; - int last_size, sum = 0; + int last_size; for (int i = 0; i < s->cu_height; i++) s->slice[i].sign = get_bits1(gb); - s->slice[0].size = last_size = sum = get_bits_long(gb, nbits); + s->slice[0].size = last_size = get_bits_long(gb, nbits); - if (sum < 0) + if (last_size < 0) return AVERROR_INVALIDDATA; for (int i = 1; i < s->cu_height; i++) { @@ -409,7 +409,6 @@ static int read_slice_sizes(RV60Context *s, GetBitContext *gb) if (last_size <= 0) return AVERROR_INVALIDDATA; s->slice[i].size = last_size; - sum += s->slice[i].size; } align_get_bits(gb); -- 2.47.2 -- Peter (A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B) [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 195 bytes --] [-- Attachment #2: Type: text/plain, Size: 251 bytes --] _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
reply other threads:[~2025-06-22 4:33 UTC|newest] Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=4cf47707d29878f27286220f42481f8651f8ed50.1750566661.git.pross@xvid.org \ --to=pross@xvid.org \ --cc=ffmpeg-devel@ffmpeg.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel This inbox may be cloned and mirrored by anyone: git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \ ffmpegdev@gitmailbox.com public-inbox-index ffmpegdev Example config snippet for mirrors. AGPL code for this site: git clone https://public-inbox.org/public-inbox.git