Skip to content

Conversation

@tarepan
Copy link
Collaborator

@tarepan tarepan commented Apr 24, 2025

内容

コア読み込みの条件分岐を共通化するリファクタリングを提案します。

他 PR のレビュー過程でコアの読み込み周りがネストで読みづらいとの指摘があった。
#1645 によりネストの1部が解消したため、これに引き続き、条件分岐の整理が求められる。

このような背景から、コア読み込みの条件分岐を共通化するリファクタリングを提案します。

関連 Issue

ref #1645

その他

「フォールバックっぽいが配置がよくわからないネスト」が残ったのですが、フォールバックの場合は順序変更が挙動を変えてしまうため手つかずです(ネスト完全解消ならず)。
そのため TODO にメモを残しています。
もし @Hiroshiba さんがここの詳細をご存知でしたらレビュー工程中で更に修正を加えたいと思います。なにかご存知でしょうか?

@tarepan tarepan requested a review from a team as a code owner April 24, 2025 18:21
@tarepan tarepan requested review from Hiroshiba and removed request for a team April 24, 2025 18:21
@tarepan
Copy link
Collaborator Author

tarepan commented Apr 29, 2025

頂いた情報を元にコード内容を再検討したところ、ネストの削除が可能と判明したため、追加リファクタリングをおこないました。これによりネストが解消します。


@Hiroshiba
全指摘箇所の反映・テストパスを確認しました。Re-review よろしくお願いします。

@takana-v takana-v requested review from Hiroshiba and takana-v May 7, 2025 03:48
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

LGTM!!!

リファクタリングありがとうございました!!!
ぜひ引き続き・・・!!!

@Hiroshiba Hiroshiba enabled auto-merge May 10, 2025 01:39
@Hiroshiba Hiroshiba added this pull request to the merge queue May 10, 2025
Merged via the queue into VOICEVOX:master with commit bb8e062 May 10, 2025
5 checks passed
@tarepan tarepan deleted the refactor/core_load_before_012 branch May 10, 2025 11:17
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.

3 participants