Bug: Projected winrt::consume_ methods will nullptr crash if the underlying QueryInterface call fails#1442

Merged

dmachaj merged 9 commits intomasterfrom user/dmachaj/failfast-bad-qi2

Oct 30, 2024

+106-236

Commits on Oct 26, 2024

Squash to a single commit
dmachaj
committed

Commits on Oct 28, 2024

Commits on Oct 29, 2024

The tests discovered a real bug. The WINRT_IMPL_SHIM macro only keeps the temporary alive until the semicolon. Remove that macro from where I need to call the check method to fix this crash
dmachaj
committed
Defragment scratch.vcxproj and get it to use cppwinrt props files as if it were a nuget package
dmachaj
committed
Move check_cast_result to impl namespace. Throw an hresult_error directly taking ownership of error info instead of re-originating
dmachaj
committed
Merge remote-tracking branch 'origin/master' into user/dmachaj/failfast-bad-qi2
dmachaj
committed
Use a reference for the cast result to avoid a spurious addref/release
dmachaj
committed
East const, add newline to end of file
dmachaj
committed