Skip to content

Fix em dash and apostrophe presentation in the Swift man page. #14719

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

Merged
merged 1 commit into from
Feb 20, 2018
Merged

Fix em dash and apostrophe presentation in the Swift man page. #14719

merged 1 commit into from
Feb 20, 2018

Conversation

krilnon
Copy link
Member

@krilnon krilnon commented Feb 19, 2018

Terminal (man swift) currently renders em dashes and apostrophes incorrectly (as the letter X rather than the proper symbol):

please file a report to AppleXs bug reporter
features X being fast is rare

This PR replaces those characters with their ASCII-safe representations. If there's a nice way to use the semantically-correct characters in the POD format, I'd be happy to use that way, too, but this fix is better than the current situation.


This PR makes a small change nearby in the same document: it reformats some whitespace to remove a double space after a sentence:

a great way for anyone to help improve Swift. The bug tracker

Terminal currently renders em dashes and apostrophes incorrectly:

"please file a report to AppleXs bug reporter"
"features X being fast is rare"

Also reformats some whitespace to remove a double space after a
sentence:

"a great way for anyone to help improve Swift.  The bug tracker"
@krilnon
Copy link
Member Author

krilnon commented Feb 19, 2018

@swift-ci Please smoke test

@krilnon krilnon requested review from natecook1000 and xwu February 19, 2018 23:56
@krilnon
Copy link
Member Author

krilnon commented Feb 19, 2018

@natecook1000 or @xwu sanity check appreciated, but I think this is fairly straightforward. It didn't break or add new warnings to podchecker either.

@xwu
Copy link
Collaborator

xwu commented Feb 20, 2018

LGTM

@krilnon krilnon merged commit c4ddd22 into swiftlang:master Feb 20, 2018
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.

3 participants