[FFmpeg-devel] [PATCH] MLP/TrueHD decoder

Michael Niedermayer michaelni
Wed Nov 14 11:28:15 CET 2007


On Wed, Nov 14, 2007 at 09:06:20AM +0000, Ian Caulfield wrote:
> Hi,
> 
> On Nov 14, 2007 2:19 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
> 
> >
> > i really dont like how the parser is implemented, not at all
> > what about:
> >
> > if(!in_sync){
> >    search for sync code, if found set remaining_size to the PREVIOUS
> > packet
> >    that is up to this sync code
> > }else if(!remaining_size){
> >    set remaining_size based on header here
> > }
> > if(remaining_size){
> >    add whats available of remaining_size into our buffer and decrease
> >    remaining_size accordingly
> > }else
> >    add everything we have into the buffer
> > if(!remaining_size)
> >    return packet
> >
> > (and yes with needed parity checks where appropriate)
> >
> 
> How would you suggest handling  the case where the sync code is split across
> two calls to the parser?

it can be split over many more than 2
and ff_combine_frame() should deal with that or did i misunderstand you
and theres another problem with that?

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

While the State exists there can be no freedom; when there is freedom there
will be no State. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071114/9b8736e8/attachment.pgp>



More information about the ffmpeg-devel mailing list