diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2017-11-24 11:24:50 +0100 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2017-11-24 15:54:48 +0100 |
commit | 832584eae514ea9cd4b3b42f1a2e9d69a17c5160 (patch) | |
tree | 98d56079c09df88b143fa3054b4797fb38fed051 /compilerplugins/clang/unnecessaryparen.cxx | |
parent | 4bae4ebd527e9deed9e1bd1be0e4bc4506d05e6f (diff) |
Clean up loplugin:unnecessaryparen blacklist
* rsc/source/parser/rscyacc.cxx no longer exists
* writerfilter/source/rtftok/rtftokenizer.cxx appears to be just fine nowadays?
* sw/source/filter/html/htmltab.cxx used redundant parentheses around a comma
operator in a while condition, and I see no reason not to remove them (the
result requires a---reasonable---tweak to loplugin:commaoperator, though)
Change-Id: I451132c700b0ae5a43b03d704156484df897ad5c
Reviewed-on: https://gerrit.libreoffice.org/45213
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'compilerplugins/clang/unnecessaryparen.cxx')
-rw-r--r-- | compilerplugins/clang/unnecessaryparen.cxx | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/compilerplugins/clang/unnecessaryparen.cxx b/compilerplugins/clang/unnecessaryparen.cxx index a7d3d26e6591..3a97c0d39456 100644 --- a/compilerplugins/clang/unnecessaryparen.cxx +++ b/compilerplugins/clang/unnecessaryparen.cxx @@ -67,14 +67,6 @@ public: return; if (loplugin::isSamePathname(fn, WORKDIR "/YaccTarget/idlc/source/parser.cxx")) return; - if (loplugin::isSamePathname(fn, WORKDIR "/YaccTarget/rsc/source/parser/rscyacc.cxx")) - return; - - // TODO yuck, comma operator at work - if (loplugin::isSamePathname(fn, SRCDIR "/writerfilter/source/rtftok/rtftokenizer.cxx")) - return; - if (loplugin::isSamePathname(fn, SRCDIR "/sw/source/filter/html/htmltab.cxx")) - return; TraverseDecl(compiler.getASTContext().getTranslationUnitDecl()); } |