[MPlayer-dev-eng] Re: [PATCH]Unicode support for ASF demuxer
Reimar Döffinger
Reimar.Doeffinger at stud.uni-karlsruhe.de
Sun Dec 3 11:15:06 CET 2006
Hello,
On Tue, Nov 28, 2006 at 04:23:07PM +0800, Zuxy Meng wrote:
[...]
forgot something:
> + if((len = contenth->title_size) != 0) {
> + wstring = (uint16_t*)&hdr[pos];
> + pos += len;
Here you use spaces.
> if (pos > hdr_len) goto len_err_out;
> - if( mp_msg_test(MSGT_HEADER,MSGL_V) )
> - print_asf_string(" Title: ", string, contenth->title_size);
> - else
> - pack_asf_string(string, contenth->title_size);
> - demux_info_add(demuxer, "name", string);
> + if (string = get_ucs2str(wstring, len)) {
> + mp_msg(MSGT_HEADER,MSGL_V," Title: %s\n", string);
> + demux_info_add(demuxer, "name", string);
> + free(string);
> + }
And here tabs.
Please try to make it consistent. I prefer spaces, but that is up to
you.
Also some compilers nowadays warn about
if (string = get_ucs2str(wstring, len))
so maybe add an extra ():
if ((string = get_ucs2str(wstring, len)))
to silence them
Greetings,
Reimar Döffinger
More information about the MPlayer-dev-eng
mailing list