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

APIドキュメントが欲しい #242

Closed
sevenc-nanashi opened this issue Aug 15, 2022 · 7 comments · Fixed by #269
Closed

APIドキュメントが欲しい #242

sevenc-nanashi opened this issue Aug 15, 2022 · 7 comments · Fixed by #269
Labels
優先度:低 初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上

Comments

@sevenc-nanashi
Copy link
Member

内容

簡易的なAPIドキュメントが欲しいです。
引数の説明や実行結果のサンプルなどが書いてあると嬉しい。

Pros 良くなる点

ラッパーライブラリを作りやすくなる

Cons 悪くなる点

ドキュメントを整備する必要が出てくる

実現方法

core.hにコメントで書いたり、docsフォルダにmdで書いたり?

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

(なし)

その他

(なし)

@Hiroshiba Hiroshiba added 機能向上 初心者歓迎タスク 初心者にも優しい簡単めなタスク 優先度:低 labels Aug 15, 2022
@Hiroshiba
Copy link
Member

Rustってなにかかっこいいドキュメント生成ツールみたいなのってあるんでしょうか?👀
(PythonのSphinxみたいな)

@qryxip
Copy link
Member

qryxip commented Aug 15, 2022

Rustのでよければrustdocですかね?
https://docs.rs/pyo3

@qryxip
Copy link
Member

qryxip commented Aug 15, 2022

rustdocの例

@qwerty2501
Copy link
Contributor

現在C API定義の改善を予定しており、ドキュメント化するのであればその後にしたほうが良さそうというのはあります。
#217

cbindgenがコメント出力できるようですが、そのときにスタイル指定できるようでDoxyとあるようです。恐らくDoxygen?
rustdoc形式で書いておけばDoxygen形式で出してくれる可能性はありそうです。
rustdoc形式だと型などがRust文脈のものなので外から見るとやや違和感があるのでできればDoxygen形式で出せそうならDoxygen形式のほうが良いと思います。

@qryxip
Copy link
Member

qryxip commented Aug 15, 2022

あーcbindgenがコメント出力できるんですね。普通に考えればコメント変換機能くらいあるか。

/**
 * TTSします。
 *
 * ```c
 * //なんかCの例
 * ```
 */
#ifdef _WIN32
__declspec(dllimport)
#endif

VoicevoxResultCode voicevox_tts(const char *text,
                                int64_t speaker_id,
                                int *output_binary_size,
                                uint8_t **output_wav);

@qwerty2501
Copy link
Contributor

お、できそうですねぇ

@qwerty2501
Copy link
Contributor

とりあえずAPIドキュメントは生成できるようになりました

まだ説明不足だったり、Doxygenの表示がイマイチだったりしますが

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.

4 participants