[vlc-commits] [Git][videolan/vlc][master] avcodec: chroma: fix for-loop condition

Steve Lhomme (@robUx4) gitlab at videolan.org
Fri Nov 10 12:31:13 UTC 2023



Steve Lhomme pushed to branch master at VideoLAN / VLC


Commits:
823d7c3a by Alexandre Janniaux at 2023-11-10T12:17:48+00:00
avcodec: chroma: fix for-loop condition

The previous `!= 0` was kept when the replacement to ARRAY_SIZE() was
done. Since <BOOL> != 0 evaluates to <BOOL>, no regression happened
because of that oversight.

Forgotten from 1eed0681cdc5da8e3ce7bc5d6b59c2658206c186.

- - - - -


1 changed file:

- modules/codec/avcodec/chroma.c


Changes:

=====================================
modules/codec/avcodec/chroma.c
=====================================
@@ -221,7 +221,7 @@ int GetVlcChroma( video_format_t *fmt, enum AVPixelFormat i_ffmpeg_chroma )
 
 enum AVPixelFormat FindFfmpegChroma( vlc_fourcc_t fourcc, bool *uv_flipped )
 {
-    for( size_t i = 0; i < ARRAY_SIZE(chroma_table) != 0; i++ )
+    for (size_t i = 0; i < ARRAY_SIZE(chroma_table); i++)
         if( chroma_table[i].i_chroma == fourcc )
         {
             *uv_flipped = fourcc == VLC_CODEC_YV12;



View it on GitLab: https://code.videolan.org/videolan/vlc/-/commit/823d7c3ae522993403492085e0ede089a2c95b8b

-- 
View it on GitLab: https://code.videolan.org/videolan/vlc/-/commit/823d7c3ae522993403492085e0ede089a2c95b8b
You're receiving this email because of your account on code.videolan.org.


VideoLAN code repository instance


More information about the vlc-commits mailing list