Skip to content

Conversation

@1uc
Copy link
Contributor

@1uc 1uc commented May 13, 2025

The missing constexpr looks like a typo, because it's present in the overview; and every other method is constexpr.

@jwakely
Copy link
Member

jwakely commented May 13, 2025

The constexpr was missing in the incoming paper, P0009R18, but I agree that it looks like an editorial mistake.

@jensmaurer
Copy link
Member

@mhoemmen , any comments here?

@mhoemmen
Copy link
Contributor

@jensmaurer Thanks for the heads-up! Thanks @1uc for pointing this out!

I agree that this must have been an editorial mistake. It was certainly the intention for required_span_size() to be marked constexpr in layout_right::mapping. It's already marked constexpr in the other two C++23 layout mappings (layout_left::mapping and layout_stride::mapping).

@jensmaurer jensmaurer merged commit 78ea606 into cplusplus:main May 15, 2025
0 of 2 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.

4 participants