[MPlayer-dev-eng] Re: [PATCH]Unicode support for ASF demuxer
Zuxy Meng
zuxy.meng at gmail.com
Sun Dec 3 06:26:02 CET 2006
Hi,
2006/12/3, Reimar Döffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de>:
> Hello,
>
> Quite nice, thanks.
>
> > #include "config.h"
> > +#include "common.h"
>
> We use "libavutil/common.h" in other places now.
OK
>
> I won't object if you want to keep it, but note that this is quite pointless
> IMO since it will crash anyway on strdup in demux_info_add, and
> printf/mp_msg is allowed (and on some systems will) crash, too.
If get_ucs2str() returns NULL, demux_info_add() won't get called and
no metadata is added.
--
Zuxy
Beauty is truth,
While truth is beauty.
PGP KeyID: E8555ED6
More information about the MPlayer-dev-eng
mailing list