Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Skeleton interface bindings #35

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Skeleton interface bindings #35

wants to merge 3 commits into from

Conversation

JeroMiya
Copy link
Contributor

The oddity you may notice in these bindings is the use of struct fields in place of the fixed sized OSVR_SkeletonJointReport arrays in the native code. Using struct fields eliminates the GC pressure from marshaling that array into a managed array (which has to be allocated and garbage collected).

However, this is a common pattern you see in many performance critical .net-native bindings. For example, XNA matrices are defined not with arrays but with public fields for each cell.

Don't merge until the OSVR-Core skeleton branch is merged:
https://github.com/OSVR/OSVR-Core/tree/skeleton

…f arrays. This ensures that the skeleton interface does not introduce GC pressure when used.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant