Skip to content
This repository has been archived by the owner on Mar 21, 2019. It is now read-only.

Support on Swift 4 #12

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Support on Swift 4 #12

wants to merge 4 commits into from

Conversation

sakatore
Copy link
Collaborator

No description provided.

@sakatore sakatore changed the title Support on Swift 4 [WIP] Support on Swift 4 Oct 21, 2017

let size = text.size(attributes: attributes)
let size = text.size(withAttributes: attributes)
let origin = textPoint(ofSize: size)
Copy link
Owner

Choose a reason for hiding this comment

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

👍

@shoheiyokoyama
Copy link
Owner

@Kyohei-Sakai
Remove [WIP] if you finished work.

@sakatore
Copy link
Collaborator Author

sakatore commented Nov 7, 2017

Remove [WIP] if you finished work.

@shoheiyokoyama
Sorry, I still haven't finished the work.

@sakatore
Copy link
Collaborator Author

@shoheiyokoyama

I changed the "Swift 3 @objc Inference" build setting to "Default" in reference to Migrate to Swift 4 @objc inference , but this warning has remained.

The use of Swift 3 @objc inference in Swift 4 mode is deprecated. Please address deprecated @objc inference warnings, test your code with “Use of deprecated Swift 3 @objc inference” logging enabled, and then disable inference by changing the "Swift 3 @objc Inference" build setting to "Default" for the "SYBlinkAnimationKit" target.

I couldn't solve the issue, so I think should tackle as other issue.

References:

@sakatore sakatore changed the title [WIP] Support on Swift 4 Support on Swift 4 Nov 21, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants