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

Add a conventional Rust layer to ittapi_rs #15

Closed
abrown opened this issue Feb 25, 2021 · 1 comment
Closed

Add a conventional Rust layer to ittapi_rs #15

abrown opened this issue Feb 25, 2021 · 1 comment

Comments

@abrown
Copy link
Contributor

abrown commented Feb 25, 2021

Working with ittapi is not the most simple thing I have done (😁) and, in order to increase adoption and speed up integration with other projects, I could use a shim layer that makes ittapi_rs use Rust conventions (e.g., it could pass around &str instead of *const c_char). What is published currently corresponds to a *-sys crate: a direct transcription of the C API to Rust. I am requesting a higher level crate built on top of that.

The intel-seapi crate had this higher-level API--see the docs. But that crate is no longer maintained. Can we add this type of functionality to ittapi_rs, potentially creating separate ittapi and ittapi-sys crates?

@abrown
Copy link
Contributor Author

abrown commented Apr 6, 2022

This is now in-tree; see #45 and #48.

@abrown abrown closed this as completed Apr 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant