Skip to content

[locale.categories] Promote remaining static const data members to constexpr #6767

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

frederick-vs-ja
Copy link
Contributor

@frederick-vs-ja frederick-vs-ja commented Jan 12, 2024

See also #4862. After the changes there's no static const data members that are guaranteed to be usable in constant expressions but not constexpr.

The changes seemingly require these static data members to be implicitly inline variables, but [contents]/1 has already required an implementation to provide definitions for them, so the difference between plain const and constexpr is unobservable.

The last remain pieces are static data members of ctype_base - it's not strictly required that these members are required to be usable in constant expressions since mask may be a class type. LWG4037 is opened for this. Edit: LWG4037 has been resolved.

@wg21bot wg21bot added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Dec 23, 2024
@frederick-vs-ja frederick-vs-ja force-pushed the locale-static-constexpr branch 2 times, most recently from 574ffea to c339cad Compare December 30, 2024 10:47
@frederick-vs-ja
Copy link
Contributor Author

Rebased & force-pushed. @tkoeppe

@frederick-vs-ja frederick-vs-ja force-pushed the locale-static-constexpr branch from c339cad to 217dc26 Compare July 18, 2025 11:15
@tkoeppe
Copy link
Contributor

tkoeppe commented Jul 18, 2025

@jwakely, @jensmaurer -- what do you think? It's effectively a no-op, but it does sort of change the normative text.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs rebase The pull request needs a git rebase to resolve merge conflicts.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants