Commit f50f149f23 for ffmpeg
commit f50f149f2325af460b210c3b31a899e9ec0414f8
Author: romainhedouin <romain.hedouin@epitech.eu>
Date: Thu Oct 15 13:02:31 2026 +0200
avcodec/j2kenc: remove an if/else with identical branches
Both branches wrote the same byte.
Found during a code review sweep of ffmpeg using an LLM.
diff --git a/libavcodec/j2kenc.c b/libavcodec/j2kenc.c
index 6493ebe499..da777580a4 100644
--- a/libavcodec/j2kenc.c
+++ b/libavcodec/j2kenc.c
@@ -279,11 +279,7 @@ static int put_cod(Jpeg2000EncoderContext *s)
// SGcod
bytestream_put_byte(&s->buf, s->prog); // progression level
bytestream_put_be16(&s->buf, s->nlayers); // num of layers
- if(s->avctx->pix_fmt == AV_PIX_FMT_YUV444P){
- bytestream_put_byte(&s->buf, 0); // unspecified
- }else{
- bytestream_put_byte(&s->buf, 0); // unspecified
- }
+ bytestream_put_byte(&s->buf, 0); // unspecified
// SPcod
bytestream_put_byte(&s->buf, codsty->nreslevels - 1); // num of decomp. levels
bytestream_put_byte(&s->buf, codsty->log2_cblk_width-2); // cblk width