Update csbindgen #40
Merged
Update csbindgen #40
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
yamachu
commented
Sep 5, 2023
| @@ -21,149 +21,117 @@ internal static unsafe partial class CoreUnsafe | |||
|
|
|||
|
|
|||
|
|
|||
| [DllImport(__DllName, EntryPoint = "initialize", CallingConvention = CallingConvention.Cdecl, ExactSpelling = true)] | |||
There was a problem hiding this comment.
globで全部持ってきたのでcompatible_engine周りが出てきてしまった
filterで対処する
yamachu
commented
Sep 5, 2023
| [DllImport(__DllName, EntryPoint = "voicevox_open_jtalk_rc_new", CallingConvention = CallingConvention.Cdecl, ExactSpelling = true)] | ||
| public static extern VoicevoxResultCode voicevox_open_jtalk_rc_new(byte* open_jtalk_dic_dir, OpenJtalkRc** out_open_jtalk); | ||
|
|
||
| [DllImport(__DllName, EntryPoint = "voicevox_open_jtalk_rc_use_user_dict", CallingConvention = CallingConvention.Cdecl, ExactSpelling = true)] | ||
| public static extern VoicevoxResultCode voicevox_open_jtalk_rc_use_user_dict(OpenJtalkRc* open_jtalk, VoicevoxUserDict* user_dict); |
There was a problem hiding this comment.
&OpenJtalkRc みたいな感じで struct を受け取ろうとしていた箇所が生成されなくなってしまった
There was a problem hiding this comment.
元々 OpenJtalkRc は
pub struct OpenJtalkRc {
open_jtalk: Arc<OpenJtalk>,
}こんな感じの定義
cbindgenとかだと
typedef struct OpenJtalkRc OpenJtalkRc;こんな感じで出力されている
FFIの際、中身をユーザに触らせないで、Pointerでやり取りするようなコードだと割りと出がちなやつ
csbindgenでも
[StructLayout(LayoutKind.Sequential)]
internal unsafe partial struct OpenJtalkRc
{
}こんな空定義が吐き出せれば良かったりする…？
There was a problem hiding this comment.
BoxやNonNullに対応してもらったお陰でだいぶ出力が増えたけど、こういう特殊なやつの対応はかなりユースケースが限られそう
一回forkリポジトリでEmitEmptyStructを許可するようなリストを読み取っていい感じにするのを試してみる
yamachu
commented
Sep 5, 2023
Comment on lines
+162
to
+172
| /// <summary>ユーザー辞書に単語を追加する。 @param [in] ユーザー辞書 @param [in] word 追加する単語 @param [out] output_word_uuid 追加した単語のUUID @returns 結果コード # Safety @param user_dict は有効な :VoicevoxUserDict のポインタであること \\safety{ - `user_dict`は ::voicevox_user_dict_new で得たものでなければならず、また ::voicevox_user_dict_delete で解放されていてはいけない。 - `word->surface`と`word->pronunciation`はヌル終端文字列を指し、かつ<a href=\"#voicevox-core-safety\">読み込みについて有効</a>でなければならない。 - `output_word_uuid`は<a href=\"#voicevox-core-safety\">書き込みについて有効</a>でなければならない。 }</summary> | ||
| [DllImport(__DllName, EntryPoint = "voicevox_user_dict_add_word", CallingConvention = CallingConvention.Cdecl, ExactSpelling = true)] | ||
| public static extern VoicevoxResultCode voicevox_user_dict_add_word(VoicevoxUserDict* user_dict, VoicevoxUserDictWord* word, void/* byte[] */* output_word_uuid); | ||
|
|
||
| /// <summary>ユーザー辞書の単語を更新する。 @param [in] user_dict ユーザー辞書 @param [in] word_uuid 更新する単語のUUID @param [in] word 新しい単語のデータ @returns 結果コード \\safety{ - `user_dict`は ::voicevox_user_dict_new で得たものでなければならず、また ::voicevox_user_dict_delete で解放されていてはいけない。 - `word_uuid`は<a href=\"#voicevox-core-safety\">読み込みについて有効</a>でなければならない。 - `word->surface`と`word->pronunciation`はヌル終端文字列を指し、かつ<a href=\"#voicevox-core-safety\">読み込みについて有効</a>でなければならない。 }</summary> | ||
| [DllImport(__DllName, EntryPoint = "voicevox_user_dict_update_word", CallingConvention = CallingConvention.Cdecl, ExactSpelling = true)] | ||
| public static extern VoicevoxResultCode voicevox_user_dict_update_word(VoicevoxUserDict* user_dict, void/* byte[] */* word_uuid, VoicevoxUserDictWord* word); | ||
|
|
||
| /// <summary>ユーザー辞書から単語を削除する。 @param [in] user_dict ユーザー辞書 @param [in] word_uuid 削除する単語のUUID @returns 結果コード \\safety{ - `user_dict`は ::voicevox_user_dict_new で得たものでなければならず、また ::voicevox_user_dict_delete で解放されていてはいけない。 - `word_uuid`は<a href=\"#voicevox-core-safety\">読み込みについて有効</a>でなければならない。 }</summary> | ||
| [DllImport(__DllName, EntryPoint = "voicevox_user_dict_remove_word", CallingConvention = CallingConvention.Cdecl, ExactSpelling = true)] | ||
| public static extern VoicevoxResultCode voicevox_user_dict_remove_word(VoicevoxUserDict* user_dict, void/* byte[] */* word_uuid); |
There was a problem hiding this comment.
&[u8; 16] のような型指定にも対応したので生成されるようになった
元々 C# 側で自前定義していたのは
yamachu
commented
Sep 6, 2023
binding/build.rs
Outdated
Comment on lines
34
to
38
| // 中身を C側 で触らない struct をここに列挙する | ||
| .rust_as_empty_struct("OpenJtalkRc") | ||
| .rust_as_empty_struct("VoicevoxSynthesizer") | ||
| .rust_as_empty_struct("VoicevoxUserDict") | ||
| .rust_as_empty_struct("VoicevoxVoiceModel") |
There was a problem hiding this comment.
このアプローチはやっぱり微妙なので、voicevox_coreのc_api側で対処するのが良さそう
There was a problem hiding this comment.
VOICEVOX/voicevox_core#512
こういう経緯があったので、voicevox_core側も厳しいかなー
csbindgenでEmptyStructを吐き出すみたいな手段が取れないだろうか
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
csbindgen 1.7.5 をベースとした https://github.com/yamachu/csbindgen/tree/support-empty-emit-struct を使ってみたらいい感じに生成できた
yamachu/csbindgen@main...yamachu:csbindgen:support-empty-emit-struct
やったことと言えば
&の部分サポート (syn::Type::Reference)2つ目はだいぶお行儀の悪いやり方なので色々とダメそう…
&のサポートを万全にしたら本家PRでもいいし、VoicevoxCoreSharpの開発だけで言えばfork先を見続けるでもいいが、悩ましいcsbindgen 1.8.0 がリリースされて、
&のサポートと、Contextとして使用するnon repr(C) のstructも出力できるようになったので、自前bindingが不要になった:clap: