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.

Sign up for GitHub

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

Jump to bottom

Support USINGZ/user data in points #6

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Support USINGZ/user data in points #6

wants to merge 2 commits into from

Conversation

msvbg
Copy link
Contributor

@msvbg msvbg commented Aug 28, 2024

This is related to #5, but I'm opening the PR here since I think it makes the most sense for the bindings to live here.

Clipper has a flag called USINGZ which adds a z field to the point structs, containing user data. This PR adds support for this USINGZ using a cargo feature flag.

Some notes:

  • I decided to pre-generate bindings both with and without USINGZ
  • I added a user_data argument to the callback, which enables a nicer API in the Rust bindings by allowing users to pass in a closure instead of a function pointer. I have another PR for this.

@msvbg msvbg mentioned this pull request Aug 28, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@msvbg