-
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
[Discussion] any plan to for high level binding? #72
Comments
Wow! That prototype covers quite a bit! Unfortunately, I don't think I have the bandwidth nor the time to drive the development of a library like that myself. That doesn't mean that I'm not interested in contributing though. Here's an alternate proposal if you're willing to drive this:
Let me know if you're interested. I do think it would be good to have some high-level bindings for XED in the rust ecosystem :) |
In fact, I'm not too satisfied with this prototype, as it's just a simple wrapper around the XED API for learning purposes. From my viewpoint, the XED API is cumbersome by design, and it looks like the implementation has been refactored a few times. We need a clearer high level binding, not a simple wrapper.
That's what I'm confused about right now, if we start a new project, what is its contribution to the community. With the few technical lines I've seen so far
It looks like our project is positioned closer to zydis.
Anything else we can add? |
I think you've got it pretty much right on library positioning: we're making xed easier to use from within rust. The benefits we provide should be the same ones that xed does and I think your list there is accurate. If we can make our hypothetical In terms of actually writing bindings that are useful here's a proposal:
This is no small amount of work :) - and all the extra to work to get it well documented will be a even more work. I do think it's worth doing though. I'm going to assume that you want to do this. I'll go ahead and create the repo, reserve the crate name, and give you access to both next weekend. That way you have a chance to at least see this message before I do anything :) |
Agree, I guess we need a If we need |
I've gone ahead and created a repo over at https://github.com/rust-xed/xed and I have also sent you an invite to join the rust-xed org. Since you said you weren't quite happy with your prototype I decided to give it a shot on my own and I think it turned out all right. I have most of the decode API wrapped - which turned out to be a rather large job - and an example on how to use it. I have translated the method documentation over but while xed is decently well documented for a C library there are still a bunch of types for which we will have to write our own documentation. |
No problem, you first according to your ideas to construct it, I will add my ideas based on your codebase, or you have any task need help, submit issue to me! |
Thank you so much for having people who are still actively maintaining this project!
I'd like to know if you all have any plan to offer high-level bindings?
In fact I did an experimental prototype a couple weeks ago, and if you're looking to start a new project I think I could get involved to contribute code!
The text was updated successfully, but these errors were encountered: