[MPlayer-dev-eng] [PATCH] demux_pva fixes

Tue Oct 11 14:47:29 CEST 2011

Hi Matteo,
Thank you for the patch.

On Monday, 10 October 2011 at 18:21, Matteo Giani wrote:
[...]
> Patch attached does 3 things:
> 
> * adds setting of needs_parsing flags
> * fixes behaviour in case a PES audio header in the source file did not
>   contain PTS field
> * some cleanup / simplification (e.g. removing unused / useless #ifdef
>   blocks which were originally there for testing back in the day, etc)

Please re-send the patch as three separate patches, each fixing one
of the above issues. Especially with the cosmetics and cleanups it's
important to have them separate. Please add [PATCH] to the subject
next time and send it to -dev-eng list only.

[...]
> Furthermore, it was suggested by Carl Eugen hat the way to go would be to
> make the ffmpeg PVA demuxer the default for these files. To this extent,
> I can confirm the ffmpeg libavformat PVA demuxer works for all the samples
> I have access to.

Thank you, this is good to know.

Regards,
Dominik

-- 
MPlayer http://mplayerhq.hu | RPM Fusion http://rpmfusion.org
There should be a science of discontent. People need hard times and
oppression to develop psychic muscles.
	-- from "Collected Sayings of Muad'Dib" by the Princess Irulan