[PATCH] D60050: Spelling correction for docs for cppcoreguidelines-owning-memory

2019-03-31 Thread Benjamin Brown via Phabricator via cfe-commits
gobennyb created this revision.
gobennyb added a reviewer: JonasToth.
gobennyb created this object with edit policy "Administrators".
gobennyb added a project: clang-tools-extra.
Herald added subscribers: cfe-commits, kbarton, nemanjai.
Herald added a project: clang.

There's a typo in the docs, as mentioned in the title. Please see the diff.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D60050

Files:
  docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst


Index: docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst
===
--- docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst
+++ docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst
@@ -50,7 +50,7 @@
   int* NonOwner = new int(42); // First warning here, since new must land in 
an owner
   delete NonOwner; // Second warning here, since only owners are allowed to be 
deleted
 
-  // Example Good, Ownership correclty stated
+  // Example Good, Ownership correctly stated
   gsl::owner Owner = new int(42); // Good
   delete Owner; // Good as well, statically enforced, that only owners get 
deleted
   


Index: docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst
===
--- docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst
+++ docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst
@@ -50,7 +50,7 @@
   int* NonOwner = new int(42); // First warning here, since new must land in an owner
   delete NonOwner; // Second warning here, since only owners are allowed to be deleted
 
-  // Example Good, Ownership correclty stated
+  // Example Good, Ownership correctly stated
   gsl::owner Owner = new int(42); // Good
   delete Owner; // Good as well, statically enforced, that only owners get deleted
   
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


[PATCH] D60050: Spelling correction for docs for cppcoreguidelines-owning-memory

2019-03-31 Thread Benjamin Brown via Phabricator via cfe-commits
gobennyb added a comment.

In D60050#1449392 , @sylvestre.ledru 
wrote:

> If you have permissions on the repo, no need for a review for such change!


I do not, so if someone could please commit for me, that'd be great. Thanks!


Repository:
  rCTE Clang Tools Extra

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60050/new/

https://reviews.llvm.org/D60050



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits