change: C APIからゲッター関数を無くす

[qryxip]

内容

VoicevoxVoiceModelFileの次のゲッター関数を、ゲッター関数ではなくする。

• voicevox_voice_model_file_id
uint8_t (*output_voice_model_id)[16]に吐き出すようにする。
• voicevox_voice_model_file_get_metas_json
voicevox_voice_model_file_create_metas_jsonに改名。

ねらいとしては #832 のような設計をC APIでも行うことで、サードパーティのバインディング製作者の労力を減らすため。上記のゲッター関数がゲッターではなくなれば、オブジェクトへのアクセスを完全に管理できる。
#849 (comment)

関連 Issue

#836

その他

CC: @yamachu

[Hiroshiba]

LGTM！！

ちょっと確認です！

確か文字列の管理は、freeが必要なやつと必要じゃないやつの2パターンあった気がします。
バージョン情報の取得とかはfree不要なタイプだった記憶。
で、そのfree不要の文字列を間違えてfreeしてしまわないように、確か何かしらの安全策を講じていた記憶があります。

今回でそのfreeしなくていいタイプの文字列が全部なくなった感じでしょうか？ 👀
それともまだ一部残ってる感じでしょうか。

これを確認している意図としては、もし全部消えた場合はその安全用のコードを削除できるなーと。
もし削除していなかったら、のちに削除が必要だな～と認識しておきたい感じです！

[qryxip]

以下の三つが、.rodataに置かれるような完全に静的な文字列を返します。

const char *voicevox_get_version(void);
const char *voicevox_get_onnxruntime_lib_versioned_filename(void);
const char *voicevox_get_onnxruntime_lib_unversioned_filename(void);

で、そのfree不要の文字列を間違えてfreeしてしまわないように、確か何かしらの安全策を講じていた記憶があります。

はい。この三つが返す静的な文字列はちゃんと追跡してるはずです。