Skip to content
This repository was archived by the owner on Jan 29, 2026. It is now read-only.

Avoid false positive in ptr_traits#367

Merged
mingxwa merged 1 commit intomicrosoft:mainfrom
mingxwa:user/mingxwa/fix-ptr-traits
Nov 17, 2025
Merged

Avoid false positive in ptr_traits#367
mingxwa merged 1 commit intomicrosoft:mainfrom
mingxwa:user/mingxwa/fix-ptr-traits

Conversation

@mingxwa
Copy link
Contributor

@mingxwa mingxwa commented Nov 15, 2025

Changes

  • Strengthened the constraints on details::ptr_traits to reject arbitrary class template instantiation.
  • Defined member type element_type in details::weak_compact_ptr to align with the updated constraints.
  • Updated the ProFacade requirements and concept proxiable accordingly.
  • Clarified the condition where the constructors/assignments of proxy participate overload resolution.
  • Added unit tests.

Resolves #366

@mingxwa mingxwa requested review from guominrui and tian-lt November 15, 2025 07:10
@mingxwa mingxwa merged commit cf7576b into microsoft:main Nov 17, 2025
10 checks passed
@mingxwa mingxwa deleted the user/mingxwa/fix-ptr-traits branch November 17, 2025 04:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

proxy does not support as a class member which would be putted into a std::variant

3 participants