[mythtv] [PATCH] XVideo XvMC VLD broken

Terry Barnaby terry1 at beam.ltd.uk
Mon May 2 19:31:07 UTC 2005


Daniel Kristjansson wrote:
> On Sun, 2005-05-01 at 12:20 +0100, Terry Barnaby wrote:
> 
>>Terry Barnaby wrote:
>>
>>>Eureka !
>>>
>>>I think I have tracked down the problem with the XvMC VLD breaking on
>>>channel changes with TS streams.
>>>
>>
>>Enclosed is an updated patch that fixes the problem of both standard
>>XvMC and XvMC VLD breaking on channel changes with TS streams.
>>
>>Note: This only handles XvMC and XvMC VLD Codecs. Other Codecs
>>that have problems with channel changes with TS streams will
>>now exit with an error message.
> 
> 
> I removed the exit, but kept the error message. One should never 
> exit from within a library, especially on data, there must be some
> proper error handling mechanism you could use.
> 
> I also didn't commit the SIPParser changes, you need to convince
> Tayler Jacob it is needed.
> 
> -- Daniel
> 
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> mythtv-dev mailing list
> mythtv-dev at mythtv.org
> http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev

Hi Daniel,

Thanks for adding the patch.
I agree there should not be an exit there, but there does not seem
to be a way of returning an error from this. I must admit, with my
searching through the libavformat and libavcodec code fro the problem 
there is some room for improvement !

Yes, the SIPParser changes were not needed, they slipped into this
patch from some one elses patch.

Now I have got the channel changing working better, (although not
ideal as there should be no need to close down and re-create Codecs,
(this could be done in mpegts) ...), I would like to get the
new merged Xv/XvMC code working with XvMC VLD.
Is it worth me having a look at this now, or shall I hang fire untill
you have completed the change to the Xv/XvMC init ?

Cheers

Terry



More information about the mythtv-dev mailing list