#84 ✓resolved
Christopher Stawarz

PythonIPCPseudoConduit::registerCallbackForCode doesn't work

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

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.

Comments and changes to this ticket

  • Christopher Stawarz
  • Christopher Stawarz

    Christopher Stawarz June 13th, 2012 @ 02:07 PM

    • Milestone set to 0.5.0
    • Milestone order changed from “168175” to “0”
  • Christopher Stawarz

    Christopher Stawarz August 27th, 2012 @ 10:40 AM

    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.

  • Christopher Stawarz
  • Christopher Stawarz

    Christopher Stawarz June 24th, 2021 @ 04:27 PM

    • State changed from “open” to “resolved”
    • Assigned user set to “Christopher Stawarz”

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.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

The core framework and supporting libraries for the MWorks Suite

People watching this ticket

Pages