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

[expr.rel] Clarify confusing wording CWG2749 #6173

Closed
wants to merge 1 commit into from

Conversation

lprv
Copy link
Contributor

@lprv lprv commented Mar 12, 2023

'Pointer to an object' is one of the pointer value categories defined in [basic.compound], but according to CWG2526 the intended reading here is that void* values are excluded from the ordering.

@jensmaurer
Copy link
Member

What's wrong with the existing text? A "pointer to void" is not a pointer to object, because void is not an object type.

@lprv
Copy link
Contributor Author

lprv commented Mar 13, 2023

The text is talking about values, and void* can hold a "pointer to object" value.

@tkoeppe
Copy link
Contributor

tkoeppe commented Apr 30, 2023

@jensmaurer Any further thoughts? This seems like a corish detail I'd defer to you on.

@jensmaurer jensmaurer added the cwg Issue must be reviewed by CWG. label Apr 30, 2023
@jensmaurer
Copy link
Member

I think this is good, but CWG should have a look because it's a bit of a delicate wording here.

@t3nsor
Copy link
Contributor

t3nsor commented Jun 13, 2023

@jensmaurer Is there a core issue for this?

@jensmaurer
Copy link
Member

CWG 2023-06-14: Needs research on C behavior for "pointer to void".

@t3nsor
Copy link
Contributor

t3nsor commented Jun 16, 2023

It appears that this is now CWG2749?

@jensmaurer jensmaurer changed the title [expr.rel] Clarify confusing wording CWG2749 [expr.rel] Clarify confusing wording Jun 16, 2023
@jensmaurer
Copy link
Member

Yes.

@jensmaurer jensmaurer changed the title CWG2749 [expr.rel] Clarify confusing wording [expr.rel] Clarify confusing wording CWG2749 Jun 17, 2023
@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 12, 2023

Already handled by CWG issue.

@tkoeppe tkoeppe closed this Nov 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cwg Issue must be reviewed by CWG.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants