[Ffmpeg-devel] [PATCH] DVR-MS probe audio format - was DVR-MS bug (MP2 decoding)

Michael Niedermayer michaelni
Sat Mar 31 21:30:28 CEST 2007


Hi

On Sat, Mar 31, 2007 at 09:43:13AM -0500, John Donaghy wrote:
> >hmm uhm, i hate dvr-ms
> 
> 
> I pretty much feel that way myself at this point but it wouldnt be so bad if
> there was decent documentation available. Anyway there's a lot of people out
> there wanting the ability to convert them so I feel it's worth this last
> push.
> 
> try av_read_frame_flush() after the codec is detected
> >and try av_seek_frame() to get back to the start
> >this is of course dirty and iam not happy about it but i dont see an easy
> >alternative, that is if this fixes it at all ...
> 
> 
> This appears to work as long as I also reset st->need_parsing to 1
> 
> I believe I can also just flush without seeking back to the start with
> almost no data loss. Is that acceptable or would you prefer that I include
> the seek as well?
> 
> In fact, now that I reset st->need_parsing I dont even really need to flush
> - I just get one or two of these messages:
> [mp3 @ 006F2F40]incorrect frame size
> 
> and then it recovers.
> 
> So do you want me to flush the stream and if so, should I also seek back to
> the start?

iam fine with either that is as long as it works, no seek no flush is of
course nicer if it mostly works

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

No snowflake in an avalanche ever feels responsible. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070331/b43cb5a1/attachment.pgp>



More information about the ffmpeg-devel mailing list