-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add a C API #39
base: master
Are you sure you want to change the base?
Add a C API #39
Conversation
Resolves #38. |
a203fa0
to
15e277b
Compare
Hmm, I should have designed this API by looking at examples/main.rs instead of Yoric/servo@70f90d0 :< |
…anges made to C API.
… through function arguments as required.
2d2ac76
to
f7b61fb
Compare
77e9710
to
f3c72d7
Compare
Thanks a lot for the PR. I'll try and review it by tomorrow. |
libc = "0" | ||
|
||
[dependencies.telemetry] | ||
path = "../" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: newline?
I'm not pulling yet as I'm discussing with the Telemetry team about changes in the histograms that would probably require several API changes. |
Yep, that makes sense. I also started to address your comments as well as support less trivial types of histograms and got stuck. |
This should make it possible to use the library from any language with a C-compatible FFI.