Closed qryxip closed 7 months ago
現在、Rust API内部でしか使われないアイテムの可視性が結構不必要にpubになっています。これらはRust APIとして普通にパブリックになります。全体的にどうなっているのかはcargo doc -p voicevox_coreで確認できます。
pub
cargo doc -p voicevox_core
不必要にpubになっているアイテムがあることは、Rust APIの公開以前に、C/Python/Java APIを含めたコードの見通しに影響しているのではないかと最近思っています。
見通しがよくなる
本当に無しかと
不必要にpubになっているものを、片っ端からpub(crate)やpub(super)にする。https://github.com/VOICEVOX/voicevox_core/pull/589#discussion_r1306609505の続きをする形。
pub(crate)
pub(super)
N/A
コアのニーズに比べて本格的に作りすぎな気がしないでもないですが、少なくとも悪いことではないと思う&設計楽しそうなので、ガシガシやっちゃっていただければ!!!
内容
現在、Rust API内部でしか使われないアイテムの可視性が結構不必要に
pub
になっています。これらはRust APIとして普通にパブリックになります。全体的にどうなっているのかはcargo doc -p voicevox_core
で確認できます。不必要に
pub
になっているアイテムがあることは、Rust APIの公開以前に、C/Python/Java APIを含めたコードの見通しに影響しているのではないかと最近思っています。Pros 良くなる点
見通しがよくなる
Cons 悪くなる点
本当に無しかと
実現方法
不必要に
pub
になっているものを、片っ端からpub(crate)
やpub(super)
にする。https://github.com/VOICEVOX/voicevox_core/pull/589#discussion_r1306609505の続きをする形。VOICEVOXのバージョン
N/A
OSの種類/ディストリ/バージョン
その他