Open hopefulGiupplo opened 10 months ago
Great work on this HostApi. The code looks very good. Just a few changes needed.
Hello @philburk, I saw you assigned this issue to yourself. Please let me know if I can help :)
Hello @philburk and PA team, I'm here to ask if I can help with any changes, and to ask if you're planning to include pa_oboe in portaudio's next release :) I rebased this work onto the last additions, I hope the library is still correctly built. Please ping me if there's anything I can do!
Thanks for rebasing this code. We are trying to get the next release out soon. This PR will need a deep review and then after it is merged it will need some soak time for other to try it. So I don't think it will make the next release.
I will try to review in depth soon. The code looks very good so far.
Hi @hopefulGiupplo, thanks for your contribution, it's looking good but I agree that Phil needs to do a thorough review since he is the Oboe expert. Some other things that would help:
Once Phil is satisfied I will also do a code review.
As I said in this issue, I finally can say that this PaOboe implementation seems to be working. Instruction to how to build PortAudio with Oboe are detailed in src/hostapi/oboe/Readme.md