Skip to content

add CallbackContext#56

Merged
teach310 merged 1 commit intodevelopfrom
feature/callback_context
Oct 28, 2023
Merged

add CallbackContext#56
teach310 merged 1 commit intodevelopfrom
feature/callback_context

Conversation

@teach310
Copy link
Copy Markdown
Owner

@teach310 teach310 commented Oct 28, 2023

Background・Issue

NativePluginから呼ばれるコールバックでエラーが発生するとUnityがクラッシュする。
ログに直接的な原因は出力されていないため原因不明

また、swiftのmainthreadでのコールバックはUnityでもmainthreadで実行されていた。

対応

Callbackが指定したSynchronizationContext.Postで実行されるようにする。

影響

  • すでにメインスレッドだった場合Delegateが呼ばれるタイミングが1フレーム遅れる。
  • swiftのメインスレッド以外でコールバックが呼ばれても大丈夫になる。

## 背景
NativePluginから呼ばれるコールバックでエラーが発生するとUnityがクラッシュする。
ログに直接的な原因は出力されていないため原因不明

また、swiftのmainthreadでのコールバックはUnityでもmainthreadで実行されていた。

## 対応
Callbackが指定したSynchronizationContext.Postで実行されるようにする。

## 影響

- すでにメインスレッドだった場合Delegateが呼ばれるタイミングが1フレーム遅れる。
- swiftのメインスレッド以外でコールバックが呼ばれても大丈夫になる。
@teach310 teach310 added the add label Oct 28, 2023
@teach310 teach310 merged commit e911052 into develop Oct 28, 2023
@teach310 teach310 deleted the feature/callback_context branch October 28, 2023 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant