Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

[firebase_ml_vision] Fix crash when scanning URL QR-code on iOS#1223

Merged
bparrishMines merged 3 commits intoflutter:masterfrom
gi097:patch-1
Mar 1, 2019
Merged

[firebase_ml_vision] Fix crash when scanning URL QR-code on iOS#1223
bparrishMines merged 3 commits intoflutter:masterfrom
gi097:patch-1

Conversation

@gi097
Copy link
Contributor

@gi097 gi097 commented Feb 14, 2019

  • NSDictionary does not accept direct null pointers, use NSNull instead
  • Other types might needs this check as well

- `NSDictionary` does not accept direct `null pointers`, use `NSNull` instead
- Other types might needs this check as well
@gi097 gi097 changed the title Fix crash when scanning URL QR-code Fix crash when scanning URL QR-code on iOS Feb 14, 2019
@bparrishMines bparrishMines changed the title Fix crash when scanning URL QR-code on iOS [firebase_ml_vision] Fix crash when scanning URL QR-code on iOS Feb 22, 2019
@bparrishMines bparrishMines added bugfix submit queue The Flutter team is in the process of landing this PR. labels Feb 22, 2019
@gi097 gi097 requested a review from bparrishMines as a code owner March 1, 2019 00:57
Copy link
Contributor

@bparrishMines bparrishMines left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks for the contribution! I'll merge once tests pass.

@bparrishMines bparrishMines merged commit f2b1e69 into flutter:master Mar 1, 2019
@gi097 gi097 deleted the patch-1 branch March 1, 2019 07:14
romaluca pushed a commit to romaluca/plugins that referenced this pull request Mar 6, 2019
julianscheel pushed a commit to jusst-engineering/plugins that referenced this pull request Mar 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bugfix cla: yes submit queue The Flutter team is in the process of landing this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants