[PATCH] Correct confusion between overwrite and override

Richard legalize at xmission.com
Thu Feb 12 21:14:12 PST 2015


Hi alexfh,

http://reviews.llvm.org/D7604

Files:
  clang-tidy/ClangTidy.h

Index: clang-tidy/ClangTidy.h
===================================================================
--- clang-tidy/ClangTidy.h
+++ clang-tidy/ClangTidy.h
@@ -82,17 +82,17 @@
 
 /// \brief Base class for all clang-tidy checks.
 ///
-/// To implement a \c ClangTidyCheck, write a subclass and overwrite some of the
+/// To implement a \c ClangTidyCheck, write a subclass and override some of the
 /// base class's methods. E.g. to implement a check that validates namespace
-/// declarations, overwrite \c registerMatchers:
+/// declarations, override \c registerMatchers:
 ///
 /// \code
 /// registerMatchers(ast_matchers::MatchFinder *Finder) {
 ///   Finder->addMatcher(namespaceDecl().bind("namespace"), this);
 /// }
 /// \endcode
 ///
-/// and then overwrite \c check(const MatchResult &Result) to do the actual
+/// and then override \c check(const MatchResult &Result) to do the actual
 /// check for each match.
 ///
 /// A new \c ClangTidyCheck instance is created per translation unit.
@@ -115,13 +115,13 @@
 
   virtual ~ClangTidyCheck() {}
 
-  /// \brief Overwrite this to register \c PPCallbacks with \c Compiler.
+  /// \brief Override this to register \c PPCallbacks with \c Compiler.
   ///
   /// This should be used for clang-tidy checks that analyze preprocessor-
   /// dependent properties, e.g. the order of include directives.
   virtual void registerPPCallbacks(CompilerInstance &Compiler) {}
 
-  /// \brief Overwrite this to register ASTMatchers with \p Finder.
+  /// \brief Override this to register ASTMatchers with \p Finder.
   ///
   /// This should be used by clang-tidy checks that analyze code properties that
   /// dependent on AST knowledge.

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D7604.19870.patch
Type: text/x-patch
Size: 1679 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150213/af16515a/attachment.bin>


More information about the cfe-commits mailing list