Skip to content

remove throw on callback#60

Merged
teach310 merged 1 commit intodevelopfrom
fix/remove_throw_on_callback
Oct 30, 2023
Merged

remove throw on callback#60
teach310 merged 1 commit intodevelopfrom
fix/remove_throw_on_callback

Conversation

@teach310
Copy link
Copy Markdown
Owner

コールバックでcontext変えるまえにエラー投げるとUnityがクラッシュするため。

DidDiscoverCharacteristicsはCharacteristicが見つからなかった場合も呼ばれて空が渡される仕様なためそもそもthrowすべきでない。 他はおそらく空なことはないため異常だとは思うが、空の配列が渡った先でthrowする必要性は感じなかったためそのまま削除

コールバックでcontext変えるまえにエラー投げるとUnityがクラッシュするため。

DidDiscoverCharacteristicsはCharacteristicが見つからなかった場合も呼ばれて空が渡される仕様なためそもそもthrowすべきでない。
他はおそらく空なことはないため異常だとは思うが、空の配列が渡った先でthrowする必要性は感じなかったためそのまま削除
@teach310 teach310 added the fix label Oct 30, 2023
@teach310 teach310 merged commit 4763dfa into develop Oct 30, 2023
@teach310 teach310 deleted the fix/remove_throw_on_callback branch October 30, 2023 13:57
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