Skip to content

Patches for GSLv4.2.2#1243

Merged
carsonRadtke merged 2 commits into
rel/4.2from
carsonradtke/gh1242
May 20, 2026
Merged

Patches for GSLv4.2.2#1243
carsonRadtke merged 2 commits into
rel/4.2from
carsonradtke/gh1242

Conversation

@carsonRadtke
Copy link
Copy Markdown
Member

Fixes #1231
Fixes #1242

The motivation of this patch is to get rid of the C4875 warnings in the latest versions of the MSVC toolset.

This change is a cherry-pick of #1213 and #1226 (authored by @beinhaerter) and will be released as GSL version 4.2.2.

carsonRadtke and others added 2 commits May 18, 2026 12:20
A new Visual Studio version will soon be available that deprecates the
old syntax for gsl::suppress. Customers will now get a C4875 diagnostic
on suppressions that look like `gsl::suppress(x)` urging them to use
`gsl::suppress("x")` instead.

This change updates the `GSL_SUPPRESS` macro to preprocess
GSL_SUPPRESS(x) to gsl::suppress("x") on clang and new versions of MSVC.
PR #1213 changed this line. According to https://github.com/microsoft/GSL/pull/1213/files#r2586073058 the change was unintended. This PR reverts the change to the previous implementation.

Co-authored-by: Werner Henze <w.henze@avm.de>
@carsonRadtke carsonRadtke merged commit d5097b0 into rel/4.2 May 20, 2026
4 checks passed
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.

3 participants