Skip to content

[Interop] [SwiftToCxx] Modify the swift::Error Dynamic Cast to return a Swift::Optional #62197

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

Robertorosmaninho
Copy link
Contributor

This PR:

  • Introduces the use of Swift::Optional on the dynamic casting of swift::Error;
  • Modifies the location where the swift::Error class is defined;
  • and erases the unnecessary NaiveException.

@hyp
Copy link
Contributor

hyp commented Nov 18, 2022

@swift-ci please test

@hyp hyp added the c++ interop Feature: Interoperability with C++ label Nov 18, 2022
@Robertorosmaninho Robertorosmaninho force-pushed the cxx-interop/DynamicCastToSwiftErrorWithOptional branch from bfc8655 to fdc1d0c Compare November 19, 2022 01:16
@LucianoPAlmeida
Copy link
Contributor

@swift-ci please test

Copy link
Contributor

@hyp hyp left a comment

Choose a reason for hiding this comment

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

LGTM

@hyp
Copy link
Contributor

hyp commented Nov 29, 2022

@swift-ci please test source compatibility

@hyp hyp merged commit 5371414 into swiftlang:main Nov 29, 2022
@Robertorosmaninho Robertorosmaninho deleted the cxx-interop/DynamicCastToSwiftErrorWithOptional branch November 30, 2022 04:18
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++
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants