[linuxtv-softmpeg] Re: libsoftmpeg bug

[Michael Hunold <hunold@convergence.de>]

Hello Bryce,

On 06/06/04 03:08, Bryce Nichols wrote:
> I found a problem in libsoftmpeg concerning handling of PES data. In the
> softmpeg.c file, the header_length value is only read if the PTS or DTS
> flags are set.  Before I changed this, I had a lot of issues with the
> video.
Thanks for the patch, I applied it to CVS.

> I am working on many changes to libsoftmpeg. 
Good to hear! 8-)

> After I've done more testing
> and cleaned up my code a bit, I will post a patch.  Significant changes
> include new AV sync code that relies less on the video refresh rate for
> synchronization, auto adaptation to the frame rate of the video material
> (so it just works automatically for either PAL or NTSC formats but with
> the option to force a particular rate or specify various methods of
> determining the frame rate), and spelling and formatting corrections.  I
> also plan to make it easy to play back either TS or PS material.  I have
> now achieved smooth video with nice AV sync when playing the mpeg2 PS
> coming out of a PVR350 card.
I'm looking forward to your patches.

Please try to split up the patches into individual chunks that can be 
applied easier. I would hate to refuse a big, large patch just because 
some small thing is not handled in a way that I can live with.

> To deal with the slow stretch-blitting until I get a CLE266 chipset, I am
> using the DirectFB Blit function instead, and just keeping the window the
> same size as the video material.  Here's the patch:
Please create your patches with "diff -uraN", they're much easier to 
read then.

> Cheers
> --Bryce
CU
Michael.