-
Notifications
You must be signed in to change notification settings - Fork 10.6k
[cxx-interop] Implicitly import Cxx module
#60603
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
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 |
zoecarver
left a comment
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 Cxxin Swift code that relies on protocols from theCxxmodule, such asCxxSequence.