summaryrefslogtreecommitdiffstats
path: root/compilerplugins/clang/dodgyswitch.cxx
diff options
context:
space:
mode:
Diffstat (limited to 'compilerplugins/clang/dodgyswitch.cxx')
-rw-r--r--compilerplugins/clang/dodgyswitch.cxx78
1 files changed, 0 insertions, 78 deletions
diff --git a/compilerplugins/clang/dodgyswitch.cxx b/compilerplugins/clang/dodgyswitch.cxx
deleted file mode 100644
index d0674b3632b0..000000000000
--- a/compilerplugins/clang/dodgyswitch.cxx
+++ /dev/null
@@ -1,78 +0,0 @@
-/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
-/*
- * This file is part of the LibreOffice project.
- *
- * This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/.
- */
-
-#include <cassert>
-#include <string>
-#include <iostream>
-#include <fstream>
-#include <set>
-#include "plugin.hxx"
-
-namespace {
-
-class DodgySwitch:
- public loplugin::FilteringPlugin<DodgySwitch>
-{
-public:
- explicit DodgySwitch(loplugin::InstantiationData const & data): FilteringPlugin(data)
- {}
-
- virtual void run() override
- {
- TraverseDecl(compiler.getASTContext().getTranslationUnitDecl());
- }
-
- bool VisitDefaultStmt(DefaultStmt const * );
- bool VisitCaseStmt(CaseStmt const * );
-private:
- bool IsParentSwitch(Stmt const * );
-};
-
-bool DodgySwitch::VisitDefaultStmt(DefaultStmt const * defaultStmt)
-{
- if (ignoreLocation(defaultStmt))
- return true;
- if (!IsParentSwitch(defaultStmt))
- report(
- DiagnosticsEngine::Warning, "default statement not directly under switch",
- compat::getBeginLoc(defaultStmt))
- << defaultStmt->getSourceRange();
- return true;
-}
-
-bool DodgySwitch::VisitCaseStmt(CaseStmt const * caseStmt)
-{
- if (ignoreLocation(caseStmt))
- return true;
- if (!IsParentSwitch(caseStmt))
- {
- //parentStmt(parentStmt(caseStmt))->dump();
- report(
- DiagnosticsEngine::Warning, "case statement not directly under switch",
- compat::getBeginLoc(caseStmt))
- << caseStmt->getSourceRange();
- }
- return true;
-}
-
-bool DodgySwitch::IsParentSwitch(Stmt const * stmt)
-{
- auto parent = getParentStmt(stmt);
- if (isa<CaseStmt>(parent) || isa<DefaultStmt>(parent)) // daisy chain
- return true;
- auto compoundStmt = dyn_cast<CompoundStmt>(parent);
- if (!compoundStmt)
- return false;
- return isa<SwitchStmt>(getParentStmt(compoundStmt));
-}
-
-loplugin::Plugin::Registration< DodgySwitch > X("dodgyswitch", false);
-
-}
-/* vim:set shiftwidth=4 softtabstop=4 expandtab: */