Mailing List Archive: MythTV: Dev

Re: [mythtv-commits] Ticket #5831: Multiple simultaneous network control

Index | Next | Previous | View Threaded

xavier.hervy at bluebottle

Nov 8, 2009, 12:46 PM

Post #1 of 2 (647 views)
Permalink

Re: [mythtv-commits] Ticket #5831: Multiple simultaneous network control

MythTV wrote:
> #5831: Multiple simultaneous network control
> --------------------------------------------------------+-------------------
> Reporter: Xavier Hervy <xavier.hervy@…> | Owner: cpinkham
> Type: enhancement | Status: closed
> Priority: minor | Milestone: unknown
> Component: mythtv | Version: head
> Severity: medium | Resolution: fixed
> Mlocked: 0 |
> --------------------------------------------------------+-------------------
> Changes (by cpinkham):
>
> * status: assigned => closed
> * resolution: => fixed
>
>
> Comment:
>
> (In [22514]) Allow mythfrontend's NetworkControl interface to support
> multiple
> simultanious connections.
>
> This is mainly a patch by Xavier Hervy with some changes by me to port it
> to
> current svn trunk, fix a couple bugs, and properly handle RESPONSE
> messages
> going to the right client (which was left as a 'fixme' in the original
> patch).
>
> Closes #5831.
>
>
Hi Chris,
Thank you for finishing the patch and applying it. I am hapy it goes to 0.22

Wouldn't mind if you explain me what the missing bits were foe though.
Xavier
_______________________________________________
mythtv-dev mailing list
mythtv-dev [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev

cpinkham at bc2va

Nov 9, 2009, 2:12 PM

Post #2 of 2 (592 views)
Permalink

Re: [mythtv-commits] Ticket #5831: Multiple simultaneous network control [In reply to]

* On Sun Nov 08, 2009 at 08:46:33PM +0000, Xavier Hervy wrote:
> >#5831: Multiple simultaneous network control

> Thank you for finishing the patch and applying it. I am hapy it goes to 0.22
>
> Wouldn't mind if you explain me what the missing bits were foe though.

I don't recall exactly off the top of my head right now. I think
the big issue was with the fixme comment you had in the code concerning
the issue of knowing which network control client the response from the
player was for. I added the ClientID to the request and response so that
I could send the reply back to the correct client rather than spamming
them all.

You could compare your diff against the actual changeset to see the diffs.

--
Chris
_______________________________________________
mythtv-dev mailing list
mythtv-dev [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact Gossamer Threads