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

Revert #503 定数化されたものを関数へ戻します #557

Merged

Conversation

shigobu
Copy link
Contributor

@shigobu shigobu commented Jul 31, 2023

内容

#503 をRevertします。
Tortoise Git を使用してRevertしました。競合が発生しましたが、元の変更等と比較しながら編集し、ビルドができるようにしました。不完全なところもあるかと思いますので、確認をお願いします。

関連 Issue

close #549

その他

自動生成された、競合についてのコメントを記載します。

# Conflicts:
#	Cargo.lock
#	Cargo.toml
#	crates/voicevox_core/Cargo.toml
#	crates/voicevox_core_c_api/Cargo.toml
#	crates/voicevox_core_c_api/include/voicevox_core.h 
#	crates/voicevox_core_c_api/src/lib.rs

kasamatsu added 3 commits July 31, 2023 11:20
This reverts commit f4b502a.

# Conflicts:
#	Cargo.lock
#	Cargo.toml
#	crates/voicevox_core/Cargo.toml
#	crates/voicevox_core_c_api/Cargo.toml
#	crates/voicevox_core_c_api/include/voicevox_core.h
#	crates/voicevox_core_c_api/src/lib.rs
open_jtalk Outdated Show resolved Hide resolved
@qryxip qryxip self-requested a review July 31, 2023 05:55
Copy link
Member

@qryxip qryxip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2点ほどありますが、あとは私が巻き取る形でも大丈夫です! ひとまずapproveを出しておきます。
(この辺は#549 (comment)にちゃんと書いておくべきでした。すみません)

crates/voicevox_core_c_api/src/lib.rs Outdated Show resolved Hide resolved
crates/download/Cargo.toml Outdated Show resolved Hide resolved
qryxip added a commit to qryxip/voicevox_core that referenced this pull request Aug 1, 2023
@qryxip qryxip mentioned this pull request Aug 1, 2023
8 tasks
@shigobu
Copy link
Contributor Author

shigobu commented Aug 10, 2023

crates/voicevox_core_c_api/include/voicevox_core.h が競合しちゃってますが、これって自動生成ですよね。
マージしないと生成結果はわからないと思うのですが、どうしましょう。

@sevenc-nanashi
Copy link
Member

voicevox_core.h以外のもののConflictを解消した後generate-c-headerしてaddすればいけると思います。

shigobu added 2 commits August 10, 2023 23:29
# Conflicts:
#	crates/voicevox_core/src/voice_synthesizer.rs
#	crates/voicevox_core_c_api/include/voicevox_core.h
#	crates/voicevox_core_c_api/src/lib.rs
Copy link
Member

@qryxip qryxip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OKです! ありがとうございました!

@Hiroshiba
Copy link
Member

おっと コンフリクトが発生していそうです 🙇

shigobu added 2 commits August 13, 2023 10:55
# Conflicts:
#	crates/voicevox_core_c_api/tests/e2e/symbols.rs
#	crates/voicevox_core_c_api/tests/e2e/testcases/tts_via_audio_query.rs
@qryxip
Copy link
Member

qryxip commented Aug 13, 2023

ありがとうございます! マージします。

@qryxip qryxip merged commit 263beb4 into VOICEVOX:main Aug 13, 2023
45 checks passed
@qryxip qryxip mentioned this pull request Aug 16, 2023
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

default option系定数をC/C++以外(主にC#)から取得する場合に煩雑になる
4 participants