[Ffmpeg-devel] [PATCH] move mpegvideo parser to mpeg12.c

Michael Niedermayer michaelni
Sun Nov 5 19:23:05 CET 2006


Hi

On Sun, Nov 05, 2006 at 06:50:28PM +0100, Baptiste Coudurier wrote:
> Michael Niedermayer wrote:
> > Hi
> > 
> > On Sun, Nov 05, 2006 at 02:27:35PM +0100, Baptiste Coudurier wrote:
> >> Hi
> >>
> >> $subject
> >>
> >> Also introduce parser.h which contains definitions and prototypes for
> >> ParseContext/1, to avoid including mpegvideo.h in pnm.c for ex.
> >>
> >> Change ff_mpeg1_find_frame_end to mpeg1_find_frame_end and make it
> >> static since it is not used anywhere now.
> > 
> > ok
> > 
> > 
> >> Also change frame_rate in ParseContext to AVRational, and use
> >> ff_frame_rate_tab to avoid annoying:
> >> Seems that stream 0 comes from film source: 25.00 (25025/1001) -> 25.00
> >> (25/1)
> > 
> > i cannot review this as the related functions are moved around
> > 
> 
> Humm, there it is. I'll apply that, then apply the other.

looks ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is




More information about the ffmpeg-devel mailing list