-
Notifications
You must be signed in to change notification settings - Fork 104
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Fix nullable and IPropertySet scenarios (#1519)
* Experiment with changes to avoid nullable.value going down the helper type route * Fix build * Fix nullable structs and delegates * Fix * Fix tests * Fix issue where the class implementing the interface can be trimmed and we rely on IDIC cast to create an instance of the interface. But that doesn't work with the interface inherits generic interfaces on AOT. This addresses that by making sure there is a fallback class available to use for the interface. * Fix warning * Improvements to nullable scenario (caching and removing old code paths) * Move where we do generic type initialization for derived generic interfaces impl class * Fix issue with Interface<> missing in impl classes by converting most of the remaining ones using it to the new format * Add test * PR feedback * PR feedback for IID
- Loading branch information
1 parent
6b5e475
commit c9f467a
Showing
15 changed files
with
746 additions
and
511 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.