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

[dcl.fct] Clarify which declarations we're talking about in example. #4287

Merged
merged 1 commit into from Jun 22, 2021

Conversation

burblebee
Copy link
Contributor

This changes "These declarations" to "The declarations above" inn example so it's clear which declarations we're talking about.

@burblebee burblebee requested a review from tkoeppe October 6, 2020 01:13
@burblebee
Copy link
Contributor Author

@tkoeppe I think this worked, yes? The issue was I had to use https://github.com/burblebee/draft.git instead of git@github.com:burblebee/draft.git because my username locally is different from github's.

@tkoeppe
Copy link
Contributor

tkoeppe commented Oct 6, 2020

@burblebee: yes, looks like it worked!

Copy link
Contributor

@tkoeppe tkoeppe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK!

source/declarations.tex Outdated Show resolved Hide resolved
@jensmaurer
Copy link
Member

Add-on improvement suggestion: We should somehow say that g1 is compared against g1 etc. Maybe add "corresponding" or "respective" somewhere. As it stands, it might be misread as-if all of the declarations are mutually functionally equivalent, but they're not.

@zygoloid zygoloid added the changes requested Changes to the wording or approach have been requested and not yet applied. label Oct 18, 2020
@jensmaurer
Copy link
Member

Apparently, the Project Editor supports my change suggestion. Please augment this pull request accordingly.

@burblebee burblebee removed the changes requested Changes to the wording or approach have been requested and not yet applied. label Apr 30, 2021
@tkoeppe tkoeppe merged commit b1bb828 into cplusplus:main Jun 22, 2021
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

4 participants