[Mplayer-cvslog] CVS: main/DOCS/tech mpcf.txt,1.30,1.31

[Alex Beregszaszi]

Hi,

> > 1, packetheader is placed _after_ a startcode
> yeah, i also realized that the ordering is problematic, the current
> one is probably the worst possible ;)
for a non error resistant format it's ok :)

> > 2a, there's a byte in the beginning of a non-keyframe with a fixed
> > value 2b, move the flags byte before the packetheader in non
> > key-frames?
> yes, better then 2a.
the patch implements 2a, but it was only a hack

> > attached the patch for libavformat (it's can't be applied after your
> > changes ;)
> ill it "apply" manually
i've applied the bugfixes already

And imho let first discuss the ordering and such, before applying.

Btw, I would remove the stream_count, file_size and length_in_msec
fields from the main header. The demuxer should read startcodes and
handle all the streamheaders it has found and the file_size/len_in_msec
fields could be placed in an optional packet, or in the info packet.

-- 
Alex Beregszaszi <alex at fsn.hu>
(MPlayer Core Developer -- http://www.mplayerhq.hu/)