This is an archive of the discontinued LLVM Phabricator instance.

s/teh/the and other grammos
ClosedPublic

Authored by eax on Mar 21 2015, 11:42 PM.

Details

Summary

all those words make teh duplicate duplicate words and other mistakes hard to find

Diff Detail

Repository
rL LLVM

Event Timeline

eax updated this revision to Diff 22416.Mar 21 2015, 11:42 PM
eax retitled this revision from to s/teh/the.
eax updated this object.
eax edited the test plan for this revision. (Show Details)
eax added a reviewer: davide.
eax added a subscriber: Unknown Object (MLST).Mar 21 2015, 11:45 PM
eax updated this revision to Diff 22417.Mar 22 2015, 1:25 AM

include a variety of other grammos as well

eax retitled this revision from s/teh/the to s/teh/the and other grammos.Mar 22 2015, 1:26 AM
eax updated this object.
mkuper added a subscriber: mkuper.Mar 22 2015, 2:08 AM

Thanks! :)
A couple of minor comments.

docs/ProgrammersManual.rst
943 ↗(On Diff #22417)

I think this should have been "it is".

docs/tutorial/LangImpl7.rst
635 ↗(On Diff #22417)

As long as we're fixing grammar here, shouldn't it be "other extension"?

eax updated this revision to Diff 22436.Mar 22 2015, 1:16 PM

Per mkuper

eax added a reviewer: mkuper.Mar 22 2015, 1:17 PM
mkuper accepted this revision.Mar 23 2015, 1:02 AM
mkuper edited edge metadata.

LGTM

This revision is now accepted and ready to land.Mar 23 2015, 1:02 AM

Do you need me to commit this for you?

This revision was automatically updated to reflect the committed changes.