Skip to content

[locale.categories] Index X_base classes and their members#8434

Closed
frederick-vs-ja wants to merge 1 commit into
cplusplus:mainfrom
frederick-vs-ja:index-locale-base-members
Closed

[locale.categories] Index X_base classes and their members#8434
frederick-vs-ja wants to merge 1 commit into
cplusplus:mainfrom
frederick-vs-ja:index-locale-base-members

Conversation

@frederick-vs-ja

@frederick-vs-ja frederick-vs-ja commented Nov 6, 2025

Copy link
Copy Markdown
Contributor

Enumerators of each unscoped enumeration type are indexed as members of the enclosing class of the enumeration.

(Attempted to fix) #6080.

@frederick-vs-ja

Copy link
Copy Markdown
Contributor Author

Oh, I overlooked member types and static data members in derived classes. Is it better to handle them in the same PR?

@eisenwave eisenwave added the P2-Bug Presentational errors and omissions label Nov 6, 2025
@tkoeppe

tkoeppe commented Nov 7, 2025

Copy link
Copy Markdown
Contributor

Up to you! Adding any missing indexings is an improvement. More is better, and I'm sure these won't be the last ones!

Enumerators of each unscoped enumeration type are indexed as members of
the enclosing class of the enumeration.
@frederick-vs-ja

frederick-vs-ja commented Nov 10, 2025

Copy link
Copy Markdown
Contributor Author

The PR is closed due to some weird reason, I'll open a new one.

@frederick-vs-ja frederick-vs-ja deleted the index-locale-base-members branch November 10, 2025 01:22
@tkoeppe

tkoeppe commented Nov 10, 2025

Copy link
Copy Markdown
Contributor

Is that because your fork got rerouted? You may need to file a ticket to have it re-attached to cplusplus/draft.

@tkoeppe

tkoeppe commented Nov 10, 2025

Copy link
Copy Markdown
Contributor

I believe GitHub support have restored all affected forks, so could you take another look?

@frederick-vs-ja

Copy link
Copy Markdown
Contributor Author

Opened #8496.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P2-Bug Presentational errors and omissions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants