change: `manifest_version`を`vvm_format_version`に

VOICEVOX / voicevox_core

無料で使える中品質なテキスト読み上げソフトウェア、VOICEVOXのコア

https://voicevox.hiroshiba.jp/

MIT License

835 stars 115 forks source link

Closed qryxip closed 1 month ago

qryxip commented 1 month ago

https://github.com/VOICEVOX/voicevox_core/issues/581#issuecomment-1837168657の1.です。manifest_version (SemVer)をvvm_format_version (整数値を文字列化したもの)にします。

実装はhttps://github.com/VOICEVOX/voicevox_core/issues/581#issuecomment-1837447739で書いたやつの冗長な方を持って来ました。 (エラーメッセージにこだわらないのならenumのやつでもいいかもしれません)

sevenc-nanashi commented 1 month ago

個人的にはenumでもいいかな～って思います。「vvmの読み込みに失敗しました。Coreが対応していないバージョンを読み込もうとしたか、vvmファイルが破損しています。」みたいなメッセージを呼び出し側で出せばそれっぽくなりそう？

Hiroshiba commented 1 month ago

@sevenc-nanashi 個人的には、そのメッセージが表示されたらVVMの破損を疑う気がします。コアを使ってくれる開発者の人の時間を本質的なところに当ててほしいので、よく遭遇しそうなエラーは丁寧に作ってあげた方が良いかなと･･･！

qryxip commented 1 month ago

バージョンはなんとなく文字列にしたのですが、Cargo.lock見たら整数値だったし整数値の方がいいのかも? (追記) エディタのRuntimeInfoのfileFormatVersionも1ですよね

Hiroshiba commented 1 month ago

バージョンはなんとなく文字列にしたのですが、Cargo.lock見たら整数値だったし整数値の方がいいのかも? (追記) エディタのRuntimeInfoのfileFormatVersionも1ですよね

確かにです！！ semverがあり得る場合は文字列にした方が良いかもです！

追記：そこさえ変えればマージしていただいても大丈夫そう！

qryxip commented 1 month ago

整数にしました。 c38b9a4 (#794)