Skip to content

Update for SE-0193 #1527

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Apr 21, 2018
Merged

Update for SE-0193 #1527

merged 1 commit into from
Apr 21, 2018

Conversation

slavapestov
Copy link
Contributor

  • @_inlineable => @inlinable
  • @_versioned => @usableFromInline
  • @inlinable @usableFromInline => @inlinable

@slavapestov
Update for SE-0193
7fb76c9 
- @_inlineable => @inlinable
- @_versioned => @usableFromInline
- @inlinable @usableFromInline => @inlinable
@slavapestov slavapestov requested a review from phausler April 20, 2018 05:12
@slavapestov
Copy link
Contributor Author

@swift-ci Please test Linux

@ikesyo
Copy link
Member

ikesyo commented Apr 20, 2018 •
edited
Loading

Why not revert this? #1502 😉

@ikesyo
Copy link
Member

ikesyo commented Apr 20, 2018

Oh I’m sorry

@inlinable @usablefrominline => @inlinable

This is different from #1497

@slavapestov
Copy link
Contributor Author

@ikesyo sorry I wasn't aware you made an earlier attempt at fixing this. As you noted though my change has an additional cleanup. Now that master has been merged into swift-4.2-branch on the Swift side this PR should be good to go.

@slavapestov slavapestov merged commit a45d718 into swiftlang:master Apr 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@parkera parkera parkera approved these changes

@phausler phausler Awaiting requested review from phausler

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@slavapestov @ikesyo @parkera