Skip to content

Add a test for synthesized get-only properties in swiftinterfaces #20244

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.

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

Merged
merged 1 commit into from
Nov 5, 2018

Conversation

jrose-apple
Copy link
Contributor

Probably superfluous with all the other tests @harlanhaskins has been adding, but I wrote it and it was sitting around on my machine, so we might as well use it.

@jrose-apple
Copy link
Contributor Author

@swift-ci Please test

@harlanhaskins
Copy link
Contributor

Thanks! I don't think these are tested in the other tests.

@swift-ci
Copy link
Contributor

swift-ci commented Nov 2, 2018

Build failed
Swift Test Linux Platform
Git Sha - a7a5d83f58fa68b3dd5dfa1327eb0bf9c12c3fb9

@swift-ci
Copy link
Contributor

swift-ci commented Nov 2, 2018

Build failed
Swift Test OS X Platform
Git Sha - a7a5d83f58fa68b3dd5dfa1327eb0bf9c12c3fb9

@jrose-apple
Copy link
Contributor Author

Uh, hm, and we don't put them all on one line anymore. Oops.

@jrose-apple
Copy link
Contributor Author

@swift-ci Please smoke test

// CHECK-NEXT: public func hash(into hasher: inout Swift.Hasher)
// CHECK-NEXT: @inlinable public init?(rawValue: Swift.Int)
// CHECK-NEXT: public var rawValue: Swift.Int {
// CHECK-NEXT: @inlinable get{{$}}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a little unfortunate, I guess. (It's a synthesized body, so we don't know how to print it.)

@beccadax
Copy link
Contributor

beccadax commented Nov 3, 2018

@swift-ci please smoke test linux platform

1 similar comment
@jrose-apple
Copy link
Contributor Author

@swift-ci please smoke test linux platform

@jrose-apple
Copy link
Contributor Author

@swift-ci Please smoke test

@jrose-apple
Copy link
Contributor Author

@swift-ci Please smoke test Linux

@jrose-apple jrose-apple merged commit df0de4d into swiftlang:master Nov 5, 2018
@jrose-apple jrose-apple deleted the getting-testy branch November 5, 2018 23:38
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.

4 participants