Skip to content
This repository has been archived by the owner on May 3, 2021. It is now read-only.

Remove callbacks from client API #20

Closed
HipsterSloth opened this issue May 3, 2016 · 1 comment
Closed

Remove callbacks from client API #20

HipsterSloth opened this issue May 3, 2016 · 1 comment

Comments

@HipsterSloth
Copy link
Collaborator

From: https://github.com/cboulay/PSMoveService/wiki/PSMoveService-Road-to-V1.0
The one major change I want to make to the Clinet API is to get rid of the callback functions for the async requests and instead switch to a result polling model similar to how OpenVR.
+ Switch all async event callback functions over to poll_next_event() style
+ This is more thread safe for clients and is easier to Marshal into C#

 ```c++
        // Process OpenVR events
        vr::VREvent_t event;
        while (m_pVRSystem->PollNextEvent(&event, sizeof(event)))
        {
            processVREvent(event);
        }
  ```
HipsterSloth pushed a commit that referenced this issue May 4, 2016
* Added new poll_next_message() api to ClientPSMoveAPI
* Added callback framework to config tool App class
* Fixed up config tool and test_console_client to use the new client API
HipsterSloth added a commit that referenced this issue May 5, 2016
Refactor for issue #20 (Remove callbacks from client API)
@HipsterSloth
Copy link
Collaborator Author

Verified compilation under OS X. Merged back to master.

HipsterSloth pushed a commit that referenced this issue Nov 8, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant