Skip to content

[_ASPendingState] Make sure accessibility strings are not nil before allocating attributed strings for them #trivial#581

Merged
nguyenhuy merged 2 commits into
TextureGroup:masterfrom
nguyenhuy:HNNilLabels
Sep 27, 2017
Merged

[_ASPendingState] Make sure accessibility strings are not nil before allocating attributed strings for them #trivial#581
nguyenhuy merged 2 commits into
TextureGroup:masterfrom
nguyenhuy:HNNilLabels

Conversation

@nguyenhuy

Copy link
Copy Markdown
Member

@ghost

ghost commented Sep 25, 2017

Copy link
Copy Markdown

🚫 CI failed with log

@ghost

ghost commented Sep 25, 2017

Copy link
Copy Markdown

🚫 CI failed with log

Comment thread Tests/ASDisplayNodeTests.mm Outdated

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Switching to @available to make CI happy.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We're assuming everyone is using Xcode 9 now I guess?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

No, this is only for CI and contributors. Users aren't forced into Xcode 9.

@ghost

ghost commented Sep 25, 2017

Copy link
Copy Markdown

🚫 CI failed with log

@ghost

ghost commented Sep 25, 2017

Copy link
Copy Markdown

🚫 CI failed with log

Comment thread Tests/ASDisplayNodeTests.mm Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We're assuming everyone is using Xcode 9 now I guess?

@ghost

ghost commented Sep 26, 2017

Copy link
Copy Markdown

🚫 CI failed with log

@nguyenhuy
nguyenhuy merged commit 9e178dc into TextureGroup:master Sep 27, 2017
@nguyenhuy
nguyenhuy deleted the HNNilLabels branch September 27, 2017 10:34
bernieperez pushed a commit to AtomTickets/Texture that referenced this pull request Apr 25, 2018
…allocating attributed strings for them #trivial (TextureGroup#581)

* Make sure accessibility strings are not nil before allocating attributed strings for them

- Fix crashes caused by TextureGroup#554

* Update tests
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