Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rust API内の各アイテムの可視性を引き締める #594

Closed
3 tasks done
qryxip opened this issue Aug 31, 2023 · 1 comment · Fixed by #759
Closed
3 tasks done

Rust API内の各アイテムの可視性を引き締める #594

qryxip opened this issue Aug 31, 2023 · 1 comment · Fixed by #759
Labels

Comments

@qryxip
Copy link
Member

qryxip commented Aug 31, 2023

内容

現在、Rust API内部でしか使われないアイテムの可視性が結構不必要にpubになっています。これらはRust APIとして普通にパブリックになります。全体的にどうなっているのかはcargo doc -p voicevox_coreで確認できます。

不必要にpubになっているアイテムがあることは、Rust APIの公開以前に、C/Python/Java APIを含めたコードの見通しに影響しているのではないかと最近思っています。

Pros 良くなる点

見通しがよくなる

Cons 悪くなる点

本当に無しかと

実現方法

不必要にpubになっているものを、片っ端からpub(crate)pub(super)にする。#589 (comment)の続きをする形。

VOICEVOXのバージョン

N/A

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

  • Windows
  • macOS
  • Linux

その他

@qryxip qryxip added the 初心者歓迎タスク 初心者にも優しい簡単めなタスク label Aug 31, 2023
@qryxip qryxip changed the title Rust APIの各APIの可視性を引き締める Rust APIの各アイテムの可視性を引き締める Aug 31, 2023
@qryxip qryxip changed the title Rust APIの各アイテムの可視性を引き締める Rust API内の各アイテムの可視性を引き締める Aug 31, 2023
@Hiroshiba
Copy link
Member

Hiroshiba commented Sep 1, 2023

コアのニーズに比べて本格的に作りすぎな気がしないでもないですが、少なくとも悪いことではないと思う&設計楽しそうなので、ガシガシやっちゃっていただければ!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants