r188440 - Add a bit of explanation for FrontendActionFactory.

Stefanus Du Toit stefanus.du.toit at intel.com
Wed Aug 14 17:35:46 PDT 2013


Author: sdt
Date: Wed Aug 14 19:35:46 2013
New Revision: 188440

URL: http://llvm.org/viewvc/llvm-project?rev=188440&view=rev
Log:
Add a bit of explanation for FrontendActionFactory.

Also use the more common "derive from" in place of "extend" in another
comment.

Modified:
    cfe/trunk/include/clang/Tooling/Tooling.h

Modified: cfe/trunk/include/clang/Tooling/Tooling.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Tooling/Tooling.h?rev=188440&r1=188439&r2=188440&view=diff
==============================================================================
--- cfe/trunk/include/clang/Tooling/Tooling.h (original)
+++ cfe/trunk/include/clang/Tooling/Tooling.h Wed Aug 14 19:35:46 2013
@@ -54,6 +54,9 @@ class FrontendAction;
 namespace tooling {
 
 /// \brief Interface to generate clang::FrontendActions.
+///
+/// Having a factory interface allows, for example, a new FrontendAction to be
+/// created for each translation unit processed by ClangTool.
 class FrontendActionFactory {
 public:
   virtual ~FrontendActionFactory();
@@ -66,7 +69,7 @@ public:
 
 /// \brief Returns a new FrontendActionFactory for a given type.
 ///
-/// T must extend clang::FrontendAction.
+/// T must derive from clang::FrontendAction.
 ///
 /// Example:
 /// FrontendActionFactory *Factory =





More information about the cfe-commits mailing list