Skip to content

[cpprestsdk] Avoid stdext::checked_array_iterator.#51750

Merged
vicroms merged 6 commits into
microsoft:masterfrom
BillyONeal:cpprestsdk-msvc-1451
May 14, 2026
Merged

[cpprestsdk] Avoid stdext::checked_array_iterator.#51750
vicroms merged 6 commits into
microsoft:masterfrom
BillyONeal:cpprestsdk-msvc-1451

Conversation

@BillyONeal
Copy link
Copy Markdown
Member

@BillyONeal BillyONeal commented May 14, 2026

See also microsoft/cpprestsdk#1836

Fixes build in VS2026 18.6.0 / MSVC 14.51

vicroms
vicroms previously approved these changes May 14, 2026
@vicroms vicroms marked this pull request as draft May 14, 2026 07:31
@BillyONeal
Copy link
Copy Markdown
Member Author

At this point I think it's a bug in feature test that marking a port as failing can make it cascade with itself due to a different feature order.

@BillyONeal
Copy link
Copy Markdown
Member Author

@BillyONeal BillyONeal marked this pull request as ready for review May 14, 2026 22:00
@vicroms vicroms added the requires:tool-release An issue that has been fixed in the microsoft/vcpkg-tool repo and is waiting for a release thereof label May 14, 2026
@BillyONeal BillyONeal removed the requires:tool-release An issue that has been fixed in the microsoft/vcpkg-tool repo and is waiting for a release thereof label May 14, 2026
@BillyONeal
Copy link
Copy Markdown
Member Author

@vicroms I don't want to wait for a tool release, I just used =skip for now. It does mean that if android were to be fixed we wouldn't know, but given the likely future of cpprestsdk getting deindexed entirely soon I think that's an acceptable risk

BillyONeal added a commit to BillyONeal/vcpkg that referenced this pull request May 14, 2026
@vicroms vicroms merged commit 9bb84f0 into microsoft:master May 14, 2026
16 checks passed
@BillyONeal BillyONeal deleted the cpprestsdk-msvc-1451 branch May 18, 2026 18:58
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.

2 participants