Skip to content

RPC タイムアウトの処理を修正する#177

Merged
melpon merged 7 commits intodevelopfrom
feature/fix-rpc-timeout
Feb 13, 2026
Merged

RPC タイムアウトの処理を修正する#177
melpon merged 7 commits intodevelopfrom
feature/fix-rpc-timeout

Conversation

@melpon
Copy link
Contributor

@melpon melpon commented Feb 12, 2026 •
edited
Loading

#176 で気になったところを修正しました。

  • タイムアウトを 0 許容にした
    • タイムアウトで 0 を許容しない設計は多分一般的には無いと思うので
  • 無駄なロックを排除
    • RPC のコールバックは Unity スレッド上で発生する
  • Canceled を削除
    • DispatchEvents() 以外でコールバックをしてはいけないし、OnDisconnect() や Dispose() の後に DispatchEvents() を呼び出してもいけないので意味がない
  • RequestRpc() のタイムアウト無しのバージョンを削除し、その代わりに DefaultRpcTimeoutMillis を public にしてユーザー側で利用可能にする
    • ユーザーにタイムアウトがあることを意識させたいのと、オーバーロードは組み合わせ爆発を起こすのであまり使いたくない
  • ResponseReceived → Response にリネーム
    • Kind としては Response が良さそうだし、短いし、これでどういう意味なのか伝わると思う

torikizi
torikizi approved these changes Feb 13, 2026
View reviewed changes
Copy link
Contributor

@torikizi torikizi left a comment

Choose a reason for hiding this comment

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

修正ありがとうございました。

t-miya
t-miya approved these changes Feb 13, 2026
View reviewed changes
Copy link
Contributor

@t-miya t-miya left a comment

Choose a reason for hiding this comment

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

ありがとうございます

@melpon melpon merged commit 3e22322 into develop Feb 13, 2026
8 checks passed
@melpon melpon deleted the feature/fix-rpc-timeout branch February 13, 2026 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@t-miya t-miya t-miya approved these changes

@torikizi torikizi torikizi approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@melpon @t-miya @torikizi