jpernst / alto

Idiomatic Rust bindings for OpenAL 1.1 and extensions (including EFX).
Apache License 2.0
80 stars 28 forks source link

FFI needs to be updated to allow the correct types to be passed into methods. #29

Closed sunnystormy closed 8 years ago

sunnystormy commented 8 years ago

I've updated my code to reflect the changes in this repo. If you look at the types of data I'm passing into the ffi, you'll see that it doesn't match.

I glanced at my C++ code to verify that I was passing the right data into the methods, and everything looks right. There may still #need to be some kinks worked out in the ffi before it becomes fully functional.

Look at my audionode-rs repo to see my updates and compare.

Thank you!

jpernst commented 8 years ago

Here are the issues I've found in your lib:

Double checking with the openal guide, the ffi bindings appear to be correct.

sunnystormy commented 8 years ago

Good to know. I'll comb over everything again and make sure the right changes are implemented. I suppose there's still a bit of a learning curve for me switching over to the "Rust" way. Thanks again for all of your help!