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

What is "it" in [memory.resource.prot]/7? #695

Closed
burblebee opened this issue Apr 14, 2016 · 1 comment
Closed

What is "it" in [memory.resource.prot]/7? #695

burblebee opened this issue Apr 14, 2016 · 1 comment
Assignees

Comments

@burblebee
Copy link
Contributor

In the note:
"For a derived class \tcode{D}, a typical implementation of this function will compute \tcode{dynamic_cast<const D*>(&other)} and go no further (i.e., return \tcode{false}) if it returns \tcode{nullptr}."
what is "it"?

e-mail from Richard:

I'm guessing "it" means the dynamic_cast expression. But in that case the wording is wrong, as it's not a function call and thus cannot return a value. Seems like this is something we can handle editorially, as this is non-normative; please either file an editorial issue or a pull request.

@jwakely
Copy link
Member

jwakely commented Apr 14, 2016

Yes, it seems clear to me that "it" is the dynamic_cast expression, although the uses of "compute" and "returns" and "go no further" all make me uneasy.

How about:

For a derived class \tcode{D}, a typical implementation of this function will immediately return \tcode{false} if \tcode{dynamic_cast<const D*>(&other) == nullptr}.

Also, earlier in the note, should "match the type of this" be "match the type of \tcode{*this}" ?

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

No branches or pull requests

2 participants