Skip to content

Extend cxplat safe integer wrappers#301

Merged
Alan-Jowett merged 2 commits intomicrosoft:mainfrom
Alan-Jowett:cxplat-safe-integer-wrappers
Apr 23, 2026
Merged

Extend cxplat safe integer wrappers#301
Alan-Jowett merged 2 commits intomicrosoft:mainfrom
Alan-Jowett:cxplat-safe-integer-wrappers

Conversation

@Alan-Jowett
Copy link
Copy Markdown
Member

@Alan-Jowett Alan-Jowett commented Apr 22, 2026

Summary

extend the cxplat safe integer wrapper surface from size_t to the fixed-width integer families

  • back the winuser and winkernel implementations directly with IntSafe and NtIntSafe helpers, enabling signed helper support in those translation units
  • add regression coverage in both cxplat_test and usersim_tests, and update the checked-in project files for the .cpp wrapper sources

Testing

  • cmake --build Q:\usersim\build --config Debug --target cxplat_test cxplat_winkernel usersim_tests -- /m
  • Q:\usersim\build\bin\Debug\cxplat_test.exe [size]
  • Q:\usersim\build\bin\Debug\usersim_tests.exe [cxplat][size]

@Alan-Jowett Alan-Jowett requested a review from mtfriesen as a code owner April 22, 2026 22:33
Add fixed-width cxplat safe integer wrappers backed by IntSafe and NtIntSafe, move the wrapper translation units to C++, and add user-mode regression coverage under tests.\n\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@Alan-Jowett Alan-Jowett force-pushed the cxplat-safe-integer-wrappers branch from a73bf90 to 497285f Compare April 23, 2026 00:11
Comment thread tests/cxplat_size_test.cpp Outdated
Drop the duplicate top-level tests coverage for cxplat size helpers and keep the dedicated cxplat_test suite as the single place covering that API.\n\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Signed-off-by: Alan Jowett <alanjo@microsoft.com>
@Alan-Jowett Alan-Jowett merged commit d8de59a into microsoft:main Apr 23, 2026
9 of 25 checks passed
@Alan-Jowett Alan-Jowett deleted the cxplat-safe-integer-wrappers branch April 23, 2026 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants