unsafe impl Send for SynthesisEngineをやめる

[qryxip]

内容

#175 により原則#[deny(unsafe_code]になっているようですが、ここがまだ残っています。

voicevox_core/crates/voicevox_core/src/engine/synthesis_engine.rs

Lines 22 to 23 in 0932c2f

	#[allow(unsafe_code)]
	unsafe impl Send for SynthesisEngine {}

Pros 良くなる点

• 新たにこのリポジトリを訪れる人に無用な不安を与えずにすむようになる

Cons 悪くなる点

• ↓の方法を取る場合、依存ライブラリの数が一つ増える

実現方法

• 非Sendなオブジェクトを安全にラップするライブラリを使う

OSの種類/ディストリ/バージョン

全部

その他

[qryxip]

よく考えたらsend_wrapperは駄目ですね。実際に複数スレッドから触れる必要があります。

[qryxip]

これ無理にunsafeを消そうとするとenum VoicevoxCoreCommandをcrossbeamで…という面倒なことになりそうですね。
実際ORTとOpenJTalkの構造体ってスレッドセーフでなくてもSend相当であると見ていいですよね?

追記: enum作らなくても Box<dyn FnOnce(&mut VoicevoxCore) -> _ + Send> Box<dyn FnOnce(&mut SynthesisEngine) -> _ + Send>でもよさそう。ただそれでも面倒です。

[qryxip]

• SyncとSendトレイトで拡張可能な並行性- The Rust Programming Language 日本語版
• Send and Sync - The Rustonomicon (英語)