Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

PythonIPCPseudoConduit::registerCallbackForCode doesn't work

#84

That is, the registered callback is never called. The problem is that the registration is done with CodecAwareConduit::registerCallback, which doesn't notify the other end of the conduit that the specified event should be forwarded.

The fix is probably to add a variant of SystemEventFactory::setEventForwardingControl that takes a code rather than a name string, and then update EventStreamConduit to handle forward-by-code.

Reported by Christopher Stawarz · June 6th, 2012 @ 01:44 PM

State: resolved
Milestone: none
Assigned to: Christopher Stawarz Christopher Stawarz

Activity

  1. Christopher Stawarz
  2. Christopher Stawarz
    Christopher Stawarz
    • Milestone set to 0.5.0
    • Milestone order changed from 168175 to 0

    June 13th, 2012 @ 02:07 PM

  3. Christopher Stawarz
    Christopher Stawarz

    I should have been clearer in my original description: registerCallbackForCode doesn't have the intended effect specifically when the other end is an EventStreamConduit, which is the case for both the client scripting bridge and the serverside conduit plugin. If the remote end is another type of conduit (say, a SimpleConduit) that sends events regardless of whether the local end has asked for them, then registerCallbackForCode will work fine.

    The fix is probably to add a variant of SystemEventFactory::setEventForwardingControl that takes a code rather than a name string, and then update EventStreamConduit to handle forward-by-code.

    While I still think that this is the best way to resolve this issue, a good intermediate fix would be for CodecAwareConduit to convert code-based registrations into name-based ones. That would work for all events except those without tag names (i.e. system events), but we could at least throw an appropriate error in that case.

    August 27th, 2012 @ 10:40 AM

  4. Christopher Stawarz
  5. Christopher Stawarz
    Christopher Stawarz
    • State changed from open to resolved
    • Assigned user set to Christopher Stawarz

    June 24th, 2021 @ 04:27 PM

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.