[mythtv-commits] Re: Ticket #155: Patch to allow connections to the backend that don't block shutdown

[MythTV]

#155: Patch to allow connections to the backend that don't block shutdown
----------------------------------+-----------------------------------------
 Reporter:  mythtv at dsl.pipex.com  |        Owner:  ijr 
     Type:  patch                 |       Status:  new 
 Priority:  minor                 |    Milestone:      
Component:  mythtv                |      Version:  head
 Severity:  medium                |   Resolution:      
       Cc:                        |  
----------------------------------+-----------------------------------------
Comment (by Paul):

 >What's the uncommented ReadStringList for?

 It's been a while since I wrote that patch but I seem to remember
 while testing the patch I added some debug code to check I was getting
 the correct response from the backend. The server socked replys where in
 sync but
 I noticed that the responses from the event socked where always lagging
 one
 reply behind where they should be. Or I wasn't getting the expected reply.

 I think I traced the problem to the commented ReadStringList in
 EventSockedConnected(). We never read the reply to "ANN Playback". So the
 first time we do a ReadStringList from the event socked we get the 'OK'
 reply from the "ANN Playback".

 I've been using the patch now for about 2 months without any problems.

 Paul

-- 
Ticket URL: <http://cvs.mythtv.org/trac/ticket/155>
MythTV <http://www.mythtv.org/>
MythTV