[PATCH] s/teh/the and other grammos

Michael Kuperstein michael.m.kuperstein at intel.com
Sun Mar 22 02:08:05 PDT 2015


Thanks! :)
A couple of minor comments.


================
Comment at: docs/ProgrammersManual.rst:943
@@ -942,3 +942,3 @@
 
-Note that is is generally preferred to *not* pass strings around as ``const
+Note that is generally preferred to *not* pass strings around as ``const
 char*``'s.  These have a number of problems, including the fact that they
----------------
I think this should have been "it is".

================
Comment at: docs/tutorial/LangImpl7.rst:635
@@ -634,3 +634,3 @@
 
-Adding var/in is just like any other other extensions we made to
+Adding var/in is just like any other extensions we made to
 Kaleidoscope: we extend the lexer, the parser, the AST and the code
----------------
As long as we're fixing grammar here, shouldn't it be "other extension"?

http://reviews.llvm.org/D8514

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list