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

[RDY] Spelling Fixes #827

Closed
wants to merge 1 commit into from
Closed

[RDY] Spelling Fixes #827

wants to merge 1 commit into from

Conversation

epmatsw
Copy link
Contributor

@epmatsw epmatsw commented Jun 8, 2014

Just fixing some misspellings.

@justinmk
Copy link
Member

justinmk commented Jun 8, 2014

LGTM, thanks

@aktau
Copy link
Member

aktau commented Jun 8, 2014

Good stuff, correct spelling does add to the perceived (and actual, imho) professionalism of a project.

@epmatsw epmatsw changed the title Spelling Fixes [RFC] Spelling Fixes Jun 8, 2014
@@ -37,7 +37,7 @@ at line number 16 in file SYS$DISK:XTESTFILE

some other error somewhere here.
...........................^
%CC-W-WARRING, Sorry, but no expalnation for such an warring.
%CC-W-WARRING, Sorry, but no explanation for such a warring.
Copy link
Member

Choose a reason for hiding this comment

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

Avoid changing test input files. They're deprecated anyway.

@justinmk
Copy link
Member

Feel free to take your time and add more corrections if you want. I'll wait until you mark [RDY] before merging. Thanks!

@epmatsw
Copy link
Contributor Author

epmatsw commented Jun 11, 2014

I think I'm pretty much done. Marking as ready.

@epmatsw epmatsw changed the title [RFC] Spelling Fixes [RDY] Spelling Fixes Jun 11, 2014
@@ -1201,7 +1201,7 @@ void msg_make(char_u *arg)
}

/*
* Output the string 'str' upto a NUL character.
* Output the string 'str' up to a NUL character.
Copy link
Member

Choose a reason for hiding this comment

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

"upto" could be considered programming jargon, like "prepend" or "outdent".

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I didn't think about that. I have removed most of the 'upto'->'up to' changes, but I left a couple that seemed to be used in a non-technical sense.

@justinmk justinmk closed this in 5b3b3fd Jun 13, 2014
@justinmk
Copy link
Member

Merged. Thanks :)

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

3 participants