[pvr.hts] Fix a couple of connection issues #95

Merged
merged 2 commits into from Dec 1, 2012

Projects

None yet

4 participants

@adamsutton
Contributor

First is a simple fix, Close() not called if message length is not read due to error

Second allows the demuxer to reconnect following the server connection dropping out.

I looked at issue #6 at the weekend and did some debugging - I think the second fix will resolve this issue, I will test this evening, I get it alot on my wireless when watching live tv on my laptop.

Tested - this seems to fix issue #6

Dogchow commented Nov 30, 2012

This does not fix issue https://github.com/opdenkamp/xbmc-pvr-addons/issues/6 for me, I'm still getting the channel freezes.

@opdenkamp opdenkamp commented on the diff Nov 30, 2012
addons/pvr.hts/src/HTSPDemux.cpp
@@ -103,6 +121,9 @@ void CHTSPDemux::Abort()
DemuxPacket* CHTSPDemux::Read()
{
+ if (!CheckConnection())
+ return PVR->AllocateDemuxPacket(0);
opdenkamp
opdenkamp Nov 30, 2012 Owner

it think you better return NULL if the connection dropped, so demux stops

@opdenkamp opdenkamp merged commit e5e761c into opdenkamp:master Dec 1, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment