Hi!

Wolfram Gloger wrote:

> I was rather blind -- dvbcut does have a full mpeg audio
> parser in streamdata::audio_addpts() which I overlooked.
> [argh, why isn't it just using the one from ffmpeg,
>  or at least called parse_audio_frames :-) ?]

IMHO, it already depends too much on ffmpeg.

> But audio_addpts has a bug which regularly hits with my
> "strange" stream.
> 
> The test for an MPEG audio header just isn't sharp enough -- with my
> test stream I regularly get false hits and therefore wrong timestamps
> are generated.

I already suspected something like that.

> The patch below fixes this, please consider my earlier patch
> withdrawn.

Yep, this one already went in. I only modified is_mpa() to speed things
up a little (you can do the test with only four conditions).

-- 
Michael "Tired" Riepe <[EMAIL PROTECTED]>
X-Tired: Each morning I get up I die a little

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
DVBCUT-user mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dvbcut-user

Reply via email to