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.delete] Missing tilde in index entry? #1417

Closed
Eelis opened this issue Jan 31, 2017 · 6 comments · Fixed by #1447
Closed

[expr.delete] Missing tilde in index entry? #1417

Eelis opened this issue Jan 31, 2017 · 6 comments · Fixed by #1447
Assignees

Comments

@Eelis
Copy link
Contributor

Eelis commented Jan 31, 2017

expressions.tex#L3644 produces:

pic

@tkoeppe
Copy link
Contributor

tkoeppe commented Feb 5, 2017

Can you please describe the perceived problem here?

@jwakely
Copy link
Member

jwakely commented Feb 5, 2017

"operator , see delete" is certainly odd ... why is there a space after "operator"? Which operator is it talking about?

@jwakely
Copy link
Member

jwakely commented Feb 5, 2017

As an aside, the text that contains the index entry ([expr.delete] p8) should probably refer to operator delete not operator delete(), and operator delete[] not operator delete[]().

@Eelis
Copy link
Contributor Author

Eelis commented Feb 5, 2017

Yeah, @jwakely's concerns about the index entry are the ones I had in mind. :)

@jwakely
Copy link
Member

jwakely commented Feb 6, 2017

I don't think it's supposed to say "operator~" though, because the unary complement operator has nothing to do with deallocation functions. That character appears in a destructor, but it's not the ~ operator.

@jensmaurer
Copy link
Member

Maybe the strange "operator space" index entry came into being during my "remove tilde from index entries" operation. In any case, operator~ has nothing to do with delete, so I'm removing the index entry.

jensmaurer added a commit to jensmaurer/draft that referenced this issue Feb 8, 2017
fix naming of global deallocation functions.

Fixes cplusplus#1417.
@jensmaurer jensmaurer self-assigned this Feb 8, 2017
tkoeppe pushed a commit that referenced this issue Feb 8, 2017
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 a pull request may close this issue.

4 participants