convert some plugins to LO_CLANG_SHARED_PLUGINS

Change-Id: I7451a95377101004c7c53c918f3234415e06f555
Reviewed-on: https://gerrit.libreoffice.org/75670
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

1 files changed, 42 insertions, 10 deletions

diff --git a/compilerplugins/clang/unicodetochar.cxx b/compilerplugins/clang/unicodetochar.cxx
index e1e381ad2c71..3599d3081a9f 100644
--- a/compilerplugins/clang/unicodetochar.cxx
+++ b/compilerplugins/clang/unicodetochar.cxx
@@ -7,6 +7,8 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
 
+#ifndef LO_CLANG_SHARED_PLUGINS
+
 #include <stack>
 
 #include "check.hxx"
@@ -24,24 +26,48 @@ public:
     explicit UnicodeToChar(loplugin::InstantiationData const & data):
         FilteringPlugin(data) {}
 
-    bool TraverseCStyleCastExpr(CStyleCastExpr * expr) {
+    bool PreTraverseCStyleCastExpr(CStyleCastExpr * expr) {
         subExprs_.push(expr->getSubExpr());
-        bool ret = RecursiveASTVisitor::TraverseCStyleCastExpr(expr);
+        return true;
+    }
+    bool PostTraverseCStyleCastExpr(CStyleCastExpr *, bool ) {
         subExprs_.pop();
+        return true;
+    }
+    bool TraverseCStyleCastExpr(CStyleCastExpr * expr) {
+        PreTraverseCStyleCastExpr(expr);
+        bool ret = RecursiveASTVisitor::TraverseCStyleCastExpr(expr);
+        PostTraverseCStyleCastExpr(expr, ret);
         return ret;
     }
 
-    bool TraverseCXXStaticCastExpr(CXXStaticCastExpr * expr) {
+    bool PreTraverseCXXStaticCastExpr(CXXStaticCastExpr * expr) {
         subExprs_.push(expr->getSubExpr());
-        bool ret = RecursiveASTVisitor::TraverseCXXStaticCastExpr(expr);
+        return true;
+    }
+    bool PostTraverseCXXStaticCastExpr(CXXStaticCastExpr *, bool) {
         subExprs_.pop();
+        return true;
+    }
+    bool TraverseCXXStaticCastExpr(CXXStaticCastExpr * expr) {
+        PreTraverseCXXStaticCastExpr(expr);
+        bool ret = RecursiveASTVisitor::TraverseCXXStaticCastExpr(expr);
+        PostTraverseCXXStaticCastExpr(expr, ret);
         return ret;
     }
 
-    bool TraverseCXXFunctionalCastExpr(CXXFunctionalCastExpr * expr) {
+    bool PreTraverseCXXFunctionalCastExpr(CXXFunctionalCastExpr * expr) {
         subExprs_.push(expr->getSubExpr());
-        bool ret = RecursiveASTVisitor::TraverseCXXFunctionalCastExpr(expr);
+        return true;
+    }
+    bool PostTraverseCXXFunctionalCastExpr(CXXFunctionalCastExpr *, bool) {
         subExprs_.pop();
+        return true;
+    }
+    bool TraverseCXXFunctionalCastExpr(CXXFunctionalCastExpr * expr) {
+        PreTraverseCXXFunctionalCastExpr(expr);
+        bool ret = RecursiveASTVisitor::TraverseCXXFunctionalCastExpr(expr);
+        PostTraverseCXXFunctionalCastExpr(expr, ret);
         return ret;
     }
 
@@ -72,18 +98,24 @@ public:
         return true;
     }
 
-private:
+    bool preRun() override {
+        return compiler.getLangOpts().CPlusPlus;
+    }
+
     void run() override {
-        if (compiler.getLangOpts().CPlusPlus) {
+        if (preRun()) {
             TraverseDecl(compiler.getASTContext().getTranslationUnitDecl());
         }
     }
 
+private:
     std::stack<Expr const *> subExprs_;
 };
 
-static loplugin::Plugin::Registration<UnicodeToChar> reg("unicodetochar");
+static loplugin::Plugin::Registration<UnicodeToChar> unicodetochar("unicodetochar");
+
+} // namespace
 
-}
+#endif // LO_CLANG_SHARED_PLUGINS
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab cinoptions=b1,g0,N-s cinkeys+=0=break: */