Skip to content

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