Skip to content

(v2.x) Remove space in user-defined literal operators to fix MSVC C4996 warning - #3184

Merged
horenmar merged 1 commit into
catchorg:v2.xfrom
ZheFeng7110:fix/v2.x-user_literal-warn
Jul 28, 2026
Merged

(v2.x) Remove space in user-defined literal operators to fix MSVC C4996 warning#3184
horenmar merged 1 commit into
catchorg:v2.xfrom
ZheFeng7110:fix/v2.x-user_literal-warn

Conversation

@ZheFeng7110

Copy link
Copy Markdown

Remove the space between "" and the suffix in user-defined literal operators (e.g. operator ""_a instead of operator "" _a) to avoid MSVC deprecation warning C5311.

Description

In MSVC, operator "" _a (with a space between "" and the suffix) triggers deprecation warning C5311.

The standard-compliant form is operator ""_a (no space), as described on cppreference.

Remove the space between "" and the suffix in user-defined literal
operators (e.g. operator ""_a instead of operator "" _a) to avoid
MSVC deprecation warning C5311.
@codecov

codecov Bot commented Jul 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.08%. Comparing base (ee1450f) to head (c002f18).
⚠️ Report is 2 commits behind head on v2.x.

Additional details and impacted files
@@           Coverage Diff           @@
##             v2.x    #3184   +/-   ##
=======================================
  Coverage   90.08%   90.08%           
=======================================
  Files         113      113           
  Lines        5040     5040           
=======================================
  Hits         4540     4540           
  Misses        500      500           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@horenmar

Copy link
Copy Markdown
Member

Sure, why not.

@horenmar
horenmar merged commit 9712bc8 into catchorg:v2.x Jul 28, 2026
3 checks passed
@ZheFeng7110
ZheFeng7110 deleted the fix/v2.x-user_literal-warn branch July 28, 2026 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants