microsoft / cppwinrt

C++/WinRT
MIT License
1.67k stars 239 forks source link

Suggestion: `check_hresult_allow_bounds` should allow `error_path_not_found` #1439

Closed pkreise closed 3 weeks ago

pkreise commented 1 month ago

Version

No response

Summary

MRT's IMapView::Lookup implementation returns ERROR_PATH_NOT_FOUND if the resource lookup fails, which causes the TryLookup projection to throw for this "expected" error case. Changing the MRT implementation would break callers that depend on this behavior, so the TryLookup projection could instead handled this alongside the existing ERROR_FILE_NOT_FOUND handler that was added for this bug:

https://github.com/microsoft/cppwinrt/issues/685

Reproducible example

No response

Expected behavior

No response

Actual behavior

No response

Additional comments

No response

github-actions[bot] commented 1 month ago

This issue is stale because it has been open 10 days with no activity. Remove stale label or comment or this will be closed in 5 days.