Skip to content

BeatSaberで再スタート時にrestaartイベントを吐き出すように改変#66

Merged
denpadokei merged 3 commits intodenpadokei:masterfrom
Run-Ahiru:master
Oct 11, 2025
Merged

BeatSaberで再スタート時にrestaartイベントを吐き出すように改変#66
denpadokei merged 3 commits intodenpadokei:masterfrom
Run-Ahiru:master

Conversation

@Run-Ahiru
Copy link
Copy Markdown

BeatSaberで再スタート時にrestaartイベントを吐き出すように改変
また、ビルド時にHttpSiraStatus\Models\GamePlayDataManager.csで「未割り当てのローカル変数'beatmapEvent'が使用されました」のエラーが発生していたため対処

@denpadokei
Copy link
Copy Markdown
Owner

プルリクエストありがとうございます。
こちらの環境では「未割り当てのローカル変数'beatmapEvent'が使用されました」が出ていないのですがこれは最初にフォークした段階から出ていたエラーですか?

@Run-Ahiru
Copy link
Copy Markdown
Author

ご返信ありがとうございます。
該当のエラーはフォークした段階で発生しておりました。

@denpadokei
Copy link
Copy Markdown
Owner

なるほど、もう一つ質問ですがビルドしたVisualStudioのバージョンを教えていただけますか?

というのも私の考えとしては動いてるコードはなるべくそのままにしていた方がバグらないという考えがあるのでできればここの変更をなかったことにしたいのと、もし発生原因があればそれをコードないのコメントとして書いておきたいなというのがあります。

@Run-Ahiru
Copy link
Copy Markdown
Author

Run-Ahiru commented Oct 11, 2025

以下の環境でビルドを行いました。

Microsoft Visual Studio Community 2019 Version 16.11.47
ビルド時の警告レベルは4

動いているコードを変更したくないというのはおっしゃる通りだと存じます。
denpadokei様の環境で問題ないということは、こちらの環境が古いことが原因と思われますので、こちらの環境を構築しなおして再度ビルドを試してみます。

@Run-Ahiru
Copy link
Copy Markdown
Author

ビルド環境を最新化し、コードを元に戻した上で問題なくビルドが通ったことを確認致しました。
他不備等ございましたらご指摘ください。
コミット歴にrevertが残ってしまったので、必要であればフォークしなおして新たにプルリクを送らせていただきます。

@denpadokei
Copy link
Copy Markdown
Owner

あー、よかった。
コミット履歴はぐちゃぐちゃで問題ないのでこれはこのままマージしちゃいます。こっちでバージョン番号だけ変えてリリースしときますね。
対応ありがとうございました。

@denpadokei denpadokei merged commit aba99ca into denpadokei:master Oct 11, 2025
@Run-Ahiru
Copy link
Copy Markdown
Author

こちらこそありがとうございます!

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.

2 participants