Skip to content

Conversation

@practicalswift
Copy link
Contributor

No description provided.

@practicalswift
Copy link
Contributor Author

getLoweredContainerOfAddress(…) was added by @eeckstein back in January 2016 (6ff2f09).

The last usage of getLoweredContainerOfAddress(…) was removed by @aschwaighofer on 2016-11-16 (cd1037b).

@eeckstein + @aschwaighofer, please review :-)

@practicalswift
Copy link
Contributor Author

@swift-ci please smoke test

@practicalswift
Copy link
Contributor Author

@swift-ci please test

@slavapestov
Copy link
Contributor

This type of thing doesn't really need review. Dead code is dead code. If there's no test for it then it never worked or did anything useful in the first place.

@practicalswift
Copy link
Contributor Author

@slavapestov Excellent! Thanks for the clear guidance!

@practicalswift practicalswift merged commit ee34650 into swiftlang:master Dec 21, 2016
@swift-ci
Copy link
Contributor

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 74aaf8e
Test requested by - @practicalswift

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.

3 participants