Skip to content

Minor modifications in concurrency section of the Rust book#36957

Merged
bors merged 1 commit intorust-lang:masterfrom
nikhilshagri:docs
Oct 6, 2016
Merged

Minor modifications in concurrency section of the Rust book#36957
bors merged 1 commit intorust-lang:masterfrom
nikhilshagri:docs

Conversation

@nikhilshagri
Copy link
Contributor

@nikhilshagri nikhilshagri commented Oct 4, 2016

frewsxcv
frewsxcv reviewed Oct 4, 2016
View reviewed changes
src/doc/book/concurrency.md Outdated
Copy link
Contributor

@frewsxcv frewsxcv Oct 4, 2016

Choose a reason for hiding this comment

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

This is a very minor detail, but is it possible you could use the 'dash' character (—) instead of a 'hyphen' character (-) for this?

https://owl.english.purdue.edu/engagement/3/7/98/

@frewsxcv
Copy link
Contributor

frewsxcv commented Oct 4, 2016

Minus the punctuation change, looks good to me!

@nikhilshagri
Copy link
Contributor Author

nikhilshagri commented Oct 4, 2016

@frewsxcv Fixed it!

@frewsxcv
Copy link
Contributor

frewsxcv commented Oct 4, 2016

Strange, it looks like there are two hyphens now. Do you also see that in the diff?

@nikhilshagri
Copy link
Contributor Author

nikhilshagri commented Oct 4, 2016

It appears like that in the diff, but I think the two hyphens are automatically fused together to form a larger hyphen in the actual markdown page. I haven't checked yet.

@GuillaumeGomez
Copy link
Member

GuillaumeGomez commented Oct 4, 2016

I see them as two hyphens as well. Dark magic.

@nikhilshagri
Copy link
Contributor Author

nikhilshagri commented Oct 4, 2016

Fixed it. I was under the impression that the long dash is formed by joining the two small dashes. To rectify this, I actually copied the dash character from here and pasted it in the code because I wasn't able to find a way to type it from my keyboard 😅

@frewsxcv
Copy link
Contributor

frewsxcv commented Oct 4, 2016

Looks great, thanks! r=me

@steveklabnik
Copy link
Contributor

steveklabnik commented Oct 4, 2016

@bors: r=frewsxcv

Thanks so much @cynicaldevil !

(And yay to @frewsxcv 's first official docs review 🎊 )

@bors
Copy link
Collaborator

bors commented Oct 4, 2016

📌 Commit 335d393 has been approved by frewsxcv

sophiajt pushed a commit to sophiajt/rust that referenced this pull request Oct 6, 2016
Rollup merge of rust-lang#36957 - cynicaldevil:docs, r=frewsxcv
534a2b5 
Minor modifications in concurrency section of the Rust book

Fixes rust-lang#36939
r? @steveklabnik
@sophiajt sophiajt mentioned this pull request Oct 6, 2016
Merged
bors added a commit that referenced this pull request Oct 6, 2016
@bors
Auto merge of #37002 - jonathandturner:rollup, r=jonathandturner
ad19c32 
Rollup of 15 pull requests

- Successful merges: #36726, #36832, #36909, #36930, #36932, #36957, #36959, #36960, #36962, #36965, #36966, #36967, #36972, #36974, #36977
- Failed merges:
@bors bors merged commit 335d393 into rust-lang:master Oct 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@frewsxcv frewsxcv frewsxcv left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@steveklabnik steveklabnik

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@nikhilshagri @frewsxcv @GuillaumeGomez @steveklabnik @bors