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にprefixを付ける、返り値をVoicevoxResultCodeで統一する #125

Closed
Hiroshiba opened this issue May 5, 2022 · 3 comments
Closed

Comments

@Hiroshiba
Copy link
Member

内容

API用headerをincludeしたときに宣言済みの名前を汚してしまうので、voicevox_ prefixを付ける形で統一したいです。
また、ついでに返り値をVoicevoxResultCodeで統一することで、last_error_messageをなくしたいです。

これら2つの破壊的変更を適用するのがこのissueの目的です。

Pros 良くなる点

使いやすくなる。

Cons 悪くなる点

APIが変わる破壊的変更なので案内が必要。

その他

での議論結果のまとめをissue化しました。

@Hiroshiba
Copy link
Member Author

エラーはついでにstderrに流すようにすると良いかも

@qwerty2501
Copy link
Contributor

破壊的変更というくくりで 配信中に話してたinitializeの引数をstruct化もしたほうがよいかもです
イメージとしてはこんな感じ

typedef struct {
   // option の引数をフィールドに入れる
} VoicevoxInitializeOption;

VoicevoxInitializeOption voicevox_default_initialize_option();

VoicevoxResultCode voicevox_initialize(VoicevoxInitializeOption option);

@qwerty2501
Copy link
Contributor

これは完了扱いでよさそうなのでCloseします

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

No branches or pull requests

2 participants