[MPlayer-dev-eng] [PATCH] Automate mov to mpegps demuxer switch

Nico Sabbi nicola_sabbi at fastwebnet.it
Fri Aug 4 22:22:39 CEST 2006


John Koleszar wrote:

>Nico Sabbi wrote:
>  
>
>>>>        
>>>>
>>>attached is a cleaner patch; if demux_mov didn't suck so much an
>>>better patch could be made.
>>>If no objects I'll commit it tomorrow
>>>
>>>      
>>>
>
>  
>

committed

>Works (of course) with both my samples. For my own education, the intent of your
>change is to let the chained demuxer search only the mpeg payload, rather than
>trying to sync to it within the mov, correct? 
>
my intent is to let demux_mov  pass the payload of mov packets/frames to 
demux_mpg;
it's less efficient than letting the open() function in demux_mov return
DEMUXER_TYPE_MPEG_PS (currently not yet possible due to the structure
of demux_mov) but it's surely more correct.

>I had been trying to make that
>work, just didn't think to assign the stream id myself. Thanks for that change.
>
>John
>  
>

Do you have any longer sample?



More information about the MPlayer-dev-eng mailing list