[Libreoffice-commits] core.git: compilerplugins/clang

Stephan Bergmann sbergman at redhat.com
Wed Jun 5 14:33:00 PDT 2013


 compilerplugins/clang/pluginhandler.cxx |   13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

New commits:
commit 3d963884cbb07f4674db2285c6ca6ac2ca1bad73
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Wed Jun 5 23:32:16 2013 +0200

    Adapt UPDATE_FILES=<module> to headers being moved to include/
    
    Change-Id: I6af5236358a51a51514d646c809147db56b2464b

diff --git a/compilerplugins/clang/pluginhandler.cxx b/compilerplugins/clang/pluginhandler.cxx
index cd554aa..e0edae2 100644
--- a/compilerplugins/clang/pluginhandler.cxx
+++ b/compilerplugins/clang/pluginhandler.cxx
@@ -21,6 +21,17 @@
 This source file manages all plugin actions. It is not necessary to modify this
 file when adding new actions.
 */
+
+namespace
+{
+
+bool isPrefix( const string& prefix, const string& full)
+    {
+    return full.compare(0, prefix.size(), prefix) == 0;
+    }
+
+}
+
 namespace loplugin
 {
 
@@ -193,7 +204,7 @@ void PluginHandler::HandleTranslationUnit( ASTContext& context )
             ; // ok
         else // scope is module
             {
-            if( strncmp( modifyFile.c_str(), ( SRCDIR "/" + scope + "/" ).c_str(), ( SRCDIR "/" + scope + "/" ).size()) != 0 )
+            if( !( isPrefix( SRCDIR "/" + scope + "/", modifyFile ) || isPrefix( SRCDIR "/include/" + scope + "/", modifyFile ) ) )
                 continue;
             }
         // Warn only now, so that files not in scope do not cause warnings.


More information about the Libreoffice-commits mailing list