sdl3: fix b80eb82

The actual count should be returned **without** duplicites.

The fixed fix fixed just the content of the array but not the count.

This would trigger assertion in
video_decoder_order_output_codecs()->get_pixfmt_desc(VIDEO_CODEC_NONE).

Author: MartinPulec

src/video_codec.c

@@ -1130,7 +1130,8 @@ i444_8_to_uyvy(int width, int height, const unsigned char *in,
 
 struct pixfmt_desc get_pixfmt_desc(codec_t pixfmt)
 {
-        assert(pixfmt >= VIDEO_CODEC_FIRST && pixfmt < VIDEO_CODEC_END);
+        assert(pixfmt >= VIDEO_CODEC_FIRST);
+        assert(pixfmt <= VIDEO_CODEC_END);
         struct pixfmt_desc ret = { 0 };
         ret.depth = codec_info[pixfmt].bits_per_channel;
         ret.subsampling = codec_info[pixfmt].subsampling;

src/video_display/sdl3.c

@@ -605,16 +605,16 @@ get_ug_to_sdl_format(const struct fmt_data *supp_fmts, codec_t ug_codec)
 static int
 get_supported_pfs(const struct fmt_data *supp_fmts, codec_t *codecs)
 {
-        bool codec_set[VC_COUNT]= {};
-        int i = 0;
-        for (; supp_fmts[i].ug_codec != VC_NONE; ++i) {
+        bool codec_set[VC_COUNT] = {};
+        int count = 0;
+        for (int i = 0; supp_fmts[i].ug_codec != VC_NONE; ++i) {
                 if (codec_set[supp_fmts[i].ug_codec]) {
                         continue;
                 }
-                codecs[i] = supp_fmts[i].ug_codec;
+                codecs[count++] = supp_fmts[i].ug_codec;
                 codec_set[supp_fmts[i].ug_codec] = true;
         }
-        return i;
+        return count;
 }
 
 static void