free-audio / clap-wrapper

Wrappers for using CLAP in other plugin environments
MIT License
120 stars 20 forks source link

Issue216 - fixing Wrapper CLAP host names #218

Closed u-he-frank closed 8 months ago

baconpaul commented 8 months ago

Heya! If you have clang-format installed you can run it across the code to fix CI. If you don't I can also do this and push to your branch. Just let me know which you prefer. The change looks good to me and is a definite improvement!

baconpaul commented 8 months ago

Actually I pushed the clang format up just now since i was in the project anyway. lets see if CI passes.

@defiantnerd this looks god to me. Agree?

defiantnerd commented 8 months ago

Yes, I just would prefer to remove the prefix i from the symbol name. Apart of that, this looks very fine.

defiantnerd commented 8 months ago

fixes https://github.com/free-audio/clap-wrapper/issues/216, thanks @u-he-frank