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

[futures.shared_future] Remove duplicated word "shared" #626

Merged
merged 1 commit into from Mar 9, 2016
Merged

[futures.shared_future] Remove duplicated word "shared" #626

merged 1 commit into from Mar 9, 2016

Conversation

jwilk
Copy link
Contributor

@jwilk jwilk commented Feb 26, 2016

No description provided.

@tkoeppe
Copy link
Contributor

tkoeppe commented Mar 7, 2016

@K-ballo dug up this history: In N2997, the original wording was:

but they synchronize on the shared associated state

It seems that "associated state" was renamed to "shared state" later, and so the construction became "shared shared state".

@zygoloid
Copy link
Member

zygoloid commented Mar 8, 2016

Can we find a better word than "shared" for the first occurrence? Maybe "the common shared state"?

@K-ballo
Copy link
Contributor

K-ballo commented Mar 8, 2016

I think dropping the first "shared" should be fine, there is only (at most) one shared state involved.

@K-ballo
Copy link
Contributor

K-ballo commented Mar 9, 2016

Anthony Williams: "That looks like a simple editorial fix to me, especially since it's in a note."
Lawrence Crowl: "Agreed."

zygoloid added a commit that referenced this pull request Mar 9, 2016
[futures.shared_future] Remove duplicated word "shared"
@zygoloid zygoloid merged commit c27eada into cplusplus:master Mar 9, 2016
@jwilk jwilk deleted the spelling-futures branch March 9, 2016 15:23
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