Skip to content

Remove the L from unevaluated string literals.#4609

Merged
nibanks merged 1 commit intomicrosoft:mainfrom
BillyONeal:fix-macos-15-xcode-16
Oct 10, 2024
Merged

Remove the L from unevaluated string literals.#4609
nibanks merged 1 commit intomicrosoft:mainfrom
BillyONeal:fix-macos-15-xcode-16

Conversation

@BillyONeal
Copy link
Member

Description

This resolves build failures on macOS 15 "Sequoia" / XCode Command Line Tools 16 detected while attempting to update vcpkg's build and test lab: microsoft/vcpkg#41307

Example:

/Users/vcpkg/Data/b/msquic/src/v2.4.5-9d07a67b94.clean/src/core/quicdef.h:243:66: error: encoding prefix 'L' on an unevaluated string literal has no effect [-Werror,-Winvalid-unevaluated-string]
  243 | CXPLAT_STATIC_ASSERT(IS_POWER_OF_TWO(QUIC_MAX_RANGE_ALLOC_SIZE), L"Must be power of two");
      |                                                                  ^

Testing

None, there are no runtime changes.

Documentation

No.

This resolves build failures on macOS 15 "Sequoia" / XCode Command Line Tools 16 detected while attempting to update vcpkg's build and test lab: microsoft/vcpkg#41307

Example:

```
/Users/vcpkg/Data/b/msquic/src/v2.4.5-9d07a67b94.clean/src/core/quicdef.h:243:66: error: encoding prefix 'L' on an unevaluated string literal has no effect [-Werror,-Winvalid-unevaluated-string]
  243 | CXPLAT_STATIC_ASSERT(IS_POWER_OF_TWO(QUIC_MAX_RANGE_ALLOC_SIZE), L"Must be power of two");
      |                                                                  ^
```
@codecov
Copy link

codecov bot commented Oct 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.80%. Comparing base (c8d66e8) to head (4ce870a).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4609      +/-   ##
==========================================
- Coverage   86.98%   85.80%   -1.19%     
==========================================
  Files          56       56              
  Lines       17354    17354              
==========================================
- Hits        15096    14891     -205     
- Misses       2258     2463     +205     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@nibanks nibanks merged commit 05fce11 into microsoft:main Oct 10, 2024
@BillyONeal BillyONeal deleted the fix-macos-15-xcode-16 branch October 10, 2024 19:07
guhetier pushed a commit that referenced this pull request Oct 9, 2025
This resolves build failures on macOS 15 "Sequoia" / XCode Command Line Tools 16 detected while attempting to update vcpkg's build and test lab: microsoft/vcpkg#41307

Example:

```
/Users/vcpkg/Data/b/msquic/src/v2.4.5-9d07a67b94.clean/src/core/quicdef.h:243:66: error: encoding prefix 'L' on an unevaluated string literal has no effect [-Werror,-Winvalid-unevaluated-string]
  243 | CXPLAT_STATIC_ASSERT(IS_POWER_OF_TWO(QUIC_MAX_RANGE_ALLOC_SIZE), L"Must be power of two");
      |                                                                  ^
```
@guhetier guhetier mentioned this pull request Oct 9, 2025
guhetier added a commit that referenced this pull request Oct 10, 2025
* fix: update OpenSSL submodule configuration for macOS SDK detection (#5453)

* Remove the L from unevaluated string literals. (#4609)

This resolves build failures on macOS 15 "Sequoia" / XCode Command Line Tools 16 detected while attempting to update vcpkg's build and test lab: microsoft/vcpkg#41307

Example:

```
/Users/vcpkg/Data/b/msquic/src/v2.4.5-9d07a67b94.clean/src/core/quicdef.h:243:66: error: encoding prefix 'L' on an unevaluated string literal has no effect [-Werror,-Winvalid-unevaluated-string]
  243 | CXPLAT_STATIC_ASSERT(IS_POWER_OF_TWO(QUIC_MAX_RANGE_ALLOC_SIZE), L"Must be power of two");
      |                                                                  ^
```

---------

Co-authored-by: Fernando Areias <nando.calheirosx@gmail.com>
Co-authored-by: Billy O'Neal <bion@microsoft.com>
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