Skip to content

Conversation

@hyp
Copy link
Contributor

@hyp hyp commented Nov 9, 2023

Fixes #65736

rdar://109714059

  • Explanation:
    Swift's SourceKit didn't code complete C++ namespace members. This changes fixes that by adding a special case that checks if an enum that's being completed is a namespace and in that case it code completes its members by looking them up using Clang APIs.
  • Scope: C++ interop, member lookup of namespace members
  • Risk: Low, affects code completion lookups for C++ interop
  • Testing: Unit tests, manual testing
  • Original PR: [cxx-interop] code-complete namespace members #69648

@hyp hyp added the c++ interop Feature: Interoperability with C++ label Nov 9, 2023
@hyp hyp requested a review from a team as a code owner November 9, 2023 18:22
@hyp
Copy link
Contributor Author

hyp commented Nov 9, 2023

@swift-ci please test

@hyp hyp changed the title [cxx-interop] code-complete namespace members [5.10][cxx-interop] code-complete namespace members Nov 9, 2023
@hyp hyp added the swift 5.10 label Nov 9, 2023
@hyp hyp merged commit 9c3c1c4 into swiftlang:release/5.10 Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ interop Feature: Interoperability with C++ swift 5.10

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants