[project-vvm-async-api] extern "C"の生ポインタをABI互換のに置き換え

[qryxip]

内容

extern "C" fnの宣言において、引数の型を次のように置き換えます。これによるC ABI上のシグネチャは変化しないはずであるので、.hは変化しません。

1. &Tとして読む*const T → &T
2. &mut Tとして読む*mut T → &mut T
3. 書き込み専用の*mut T → &mut MaybeUninit<T>
4. Box<T>として扱う*mut T → Box<T>

各公式ドキュメントを読めばこの置き換えをしていいことがわかりますし、cbindgenもそのように判断しています。
また1., 2., 3.についてはOSSでの実例もいくつも見つけることができます。4.だけ実例をあまり見つけることができないのですが、これはBox<impl Sized>のレイアウトが正式に保証されたのが Rust 1.41 (2020年)と比較的最近だからだと思われます(ただこのリンク先のコード例をもって正当性の根拠とすることができると思います)。

関連 Issue

#497

その他

mainブランチの方に向けようか迷ったのですが、Synthesizerとかへの適用が主になると思ったので、project-vvm-async-apiに向けることにしました。

このPRは現在 #512 に続く形で実装しています。

[qryxip]

あ、このPRの利点ですが一応:

• 健全性(soundness)の向上
• 読むときの認知負荷の軽減

を目的としています。

[qryxip]

あ、*mut u8(wavファイルのやつ)とかもNonNull<u8>とかにしておいた方がいいかも。

[qryxip]

↑ なしで。ユーザーに貸し出すべき配列が空だったらとか考えると利点の割に面倒になります。

[qryxip]

なんかunsafeが10箇所不要になって外せました。

[Hiroshiba]

LGTM！！

引数の型でMaybeUninit使うの面白いですね。
本当にやりたいのはwrite onlyな気がしますが。（そういうの無さそう）

[Hiroshiba]

問題ないと思うのでマージします！！