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

Resolving, some but not all, GEN-ART comments. #13

Merged
merged 3 commits into from Mar 25, 2017
Merged

Conversation

seanturner
Copy link
Contributor

This deals with some of Meral's GEN-ART comments.

standardization or by other means of agreement. Existing protocols (for
example SIP <xref target="RFC3261"/> or XMPP <xref target="RFC6120"/>)
standardization or by other means of agreement. Existing protocols
(e.g. SIP <xref target="RFC3261"/> or XMPP <xref target="RFC6120"/>)
Copy link
Contributor

Choose a reason for hiding this comment

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

I have a principled objection to using Latin-based abbreviations when there are common English words that mean exactly the same thing. It provides a stumbling block for international readers. (I had to look up "e. g." to make sure there was no difference in meaning - which there isn't, according to Wiktionary).

@seanturner
Copy link
Contributor Author

I can see putting it back, but we should be consistent there were two e.g. and two for examples. My matchy-matchy tendencies getting to me.

@alvestrand
Copy link
Contributor

:-) I can flip it in the other direction. Sometimes I show off my English skills, othertimes I want to write simple English.
Trapped by my own filters .... the review ended up in my gen-art inbox, which I stopped reading years ago when I left the gen-art team....

@seanturner
Copy link
Contributor Author

seanturner commented Mar 21, 2017 via email

@alvestrand alvestrand merged commit 33ad99a into master Mar 25, 2017
@alvestrand alvestrand deleted the gen-art_nits branch March 25, 2017 17:42
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

2 participants