[MPlayer-dev-eng] ASF file's pixel aspect ratio

[Zuxy Meng]

Hello,

I'd like to introduce pixel aspect ratio support to ASF demuxer. I've
written a simple working prototype but to go further I have some
questions to ask:

1. ASF supports two types of aspect info: one is a per stream metadata
called AspectRatioX|Y, the other is a per packet payload extension.
The first is easier to implement: I simply set the proper
(stream_video_t*)sh->aspect value in asfheader.c and it's done. The
second is not so straight forward: does mplayer support non-constant
aspect ratio for a video stream? I saw in the ChangeLog of pre8 that
mpeg12 variant aspect ratio is supported. How and where should I
implement it for ASF? (And I also wonder how would it look like,
hehe.)

2. ASF makes use of ucs16le encoded string extensively. While Windows
provides many native routines to handle them, other OSes don't.
Especially for mplayer, all internal strings are expected to be
encoded as utf-8. But there isn't a utf6le->utf-8 conversion routine
available yet, although Reimar has suggested a static write_utf8() in
libavcodec/flacenc.c. If I were to implement one that might be used
for both mplayer and ffmpeg, where should I put it? In libavutil?

3. Files dealing with ASF have very strange indentation, am I free to
add something like /* vim se noet, se sw=2 */ to the end of a file?

4. Those files also lack proper copyright headers...so am I free to
modify them anyway?:-)
-- 
Zuxy
Beauty is truth,
While truth is beauty.
PGP KeyID: E8555ED6