-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
[cxx-interop] Implicitly import Cxx
module
#60603
Conversation
@swift-ci please smoke test |
8e96abd
to
2b4a7ff
Compare
@swift-ci please smoke test |
Should we rename it to |
@stevapple underscored names are generally used for something the developer wouldn't need to reference directly from Swift code. This module could be referenced directly, for example, if someone needs to provide a utility extension to one of the protocols from // Util.swift
import Cxx
extension CxxSequence where RawIterator: ... {
func doSomething() { ... }
} Note that we don't always implicitly import |
Ah, makes sense👌 |
@swift-ci please smoke test |
75858ca
to
6e61fc3
Compare
@swift-ci please smoke test |
6e61fc3
to
9e84c84
Compare
@swift-ci please smoke test |
9e84c84
to
106b7b9
Compare
@swift-ci please smoke test |
106b7b9
to
5322433
Compare
@swift-ci please smoke test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for fixing all these tests!
5322433
to
4f5552b
Compare
@swift-ci please smoke test |
1 similar comment
@swift-ci please smoke test |
4f5552b
to
4562328
Compare
@swift-ci please smoke test |
4562328
to
ca38af8
Compare
@swift-ci please smoke test |
Hooray, makes a lot of sense +1 |
ca38af8
to
1918d1d
Compare
@swift-ci please smoke test |
1918d1d
to
d0d3c03
Compare
@swift-ci please smoke test |
d0d3c03
to
cc0a8b1
Compare
@swift-ci please smoke test |
This lifts the requirement for the user to explicitly add `import Cxx` in Swift code that relies on protocols from the `Cxx` module, such as `CxxSequence`.
`Swift` is the name of the Swift standard library module. Creating another module with this name breaks assumptions in the compiler and caused these two tests to fail.
cc0a8b1
to
5a3bb14
Compare
@swift-ci please smoke test |
@swift-ci please smoke test macOS |
This lifts the requirement for the user to explicitly add
import Cxx
in Swift code that relies on protocols from theCxx
module, such asCxxSequence
.