SDL_mixer_metadata_tags.c:parse_id3v1_ansi_string(): Fixed "always true" expression

[Sackzement]

The comparison i >= 0 is always true because i is an unsigned type.

[  6%] Building C object CMakeFiles/SDL3_mixer-shared.dir/src/SDL_mixer_metadata_tags.c.o
/path/to/SDL_mixer/src/SDL_mixer_metadata_tags.c: In function ‘parse_id3v1_ansi_string’:
/path/to/SDL_mixer/src/SDL_mixer_metadata_tags.c:99:37: warning: comparison of unsigned expression in ‘>= 0’ is always true [-Wtype-limits]
   99 |     for (size_t i = src_len - 1; (i >= 0) && (src_buffer[i] == ' '); i--) {
      |                                     ^~

This commit uses a do-while loop instead.

Might be a bit more "chatty". :)
4 lines vs 12 lines

[Sackzement]

Totally forgot the break;for the second condition. It's fixed now.

[slouken]

Merged, thanks!

[sezero]

And why is the loop iterator in there is a size_t instead of a plain signed int??

[Sackzement]

And why is the loop iterator in there is a size_t instead of a plain signed int??

src_len is also size_t. The loop iterator just uses the same type as what is passed to the function.
My guess is because the buffer can be very large which int wouldn't be able to index.