[MP3 ENCODER] id3tag.c

[Adam Whitehead]

Hi,

This is hardly important, but I noticed in id3_pad() in id3tag.c, the
function
that adds padding to ID3 tags writes space characters for padding.

--snip--
/*
 * If "string" is shorter than "length", pad it with ' ' (spaces)
 */

static void id3_pad(char *string, int length) {
        int l;  l=strlen(string);

        while(l<length) { string[l] = ' '; l++; }
        string[l]='\0';
        }
--snip--

The ID3 spec asks for NULL chars to be used for padding. It's no big deal,
but it
makes testing for an ID3v1.1 track field (which is the last byte of the
comment field)
easier, because a NULL is not a valid character in the fields (except as
padding).

Regards,
Adam Whitehead
CEO - Netherworld Media
E-mail: [EMAIL PROTECTED]
Phone: +61 (889) 279 898
FAX: +61 (889) 273 889
Musicseek (http://www.musicseek.net)


--
MP3 ENCODER mailing list ( http://geek.rcc.se/mp3encoder/ )