Skip to content
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

[class.static.data] Fix - classes have no subobjects #6444

Closed
wants to merge 1 commit into from

Conversation

Eisenwave
Copy link
Contributor

To my understanding, a class cannot have any subobjects, only an object of class type can have subobjects. Therefore, the wording is incorrect, and should state class object instead of just class.

@tkoeppe
Copy link
Contributor

tkoeppe commented Aug 14, 2023

The original wording seems to have issues, but I'm not sure if this is an improvement. I think a more thorough rewording is needed here.

@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 10, 2023

We say "subobject(s) of a class" in multiple places, so let's not make a single change like this. Could you maybe start by looking for all the occurrences. (E.g. in [conv.mem] we say "the subobject of D", where D is a class.

I think "class object" is generally a phrase we can use, but we'll have to look at each individual case to see how to make it fit.

@jensmaurer
Copy link
Member

The standard is generally confused about "base class" and "data member" and "subobject" and the relationship among them. (For example, you can have multiple base classes of the same type, that may or may not ("virtual") cause different subobjects to exist in any class object.)

I don't think I'd support applying any cleanup in this area in the context of an editorial issue (as opposed to a core issue).

@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 10, 2023

OK, thanks, @jensmaurer! Let's close this issue then. @Eisenwave: please file a CWG issue if you want to pursue this.

@tkoeppe tkoeppe closed this Nov 10, 2023
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.

None yet

3 participants