summaryrefslogtreecommitdiff
path: root/idlc
diff options
context:
space:
mode:
authorStephan Bergmann <sbergman@redhat.com>2017-11-27 09:46:06 +0100
committerStephan Bergmann <sbergman@redhat.com>2017-11-27 14:04:40 +0100
commita2656563f1a7639a99276fda90b6c8f675d20fc5 (patch)
treecc9eb4f064b1e8cdfce9ac42c52dfa6ae54f2b70 /idlc
parent0093ae5f2ca1141f199aa9ae3b34b05a33def928 (diff)
loplugin:unnecessaryparen: Warn about parentheses around literals
...that are not composed of multiple tokens, like ("foo" "bar"). Also don't yet warn about Boolean literals, which are sometimes wrapped in parentheses to silence unreachable-code warnings. To avoid multiple warnings about code like f((0)) switch to generally using a set of ParenExpr to keep track of which occurrences have already been handled. Change-Id: I036a25a92836ec6ab6c56ea848f71bc6d63822bc Reviewed-on: https://gerrit.libreoffice.org/45317 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'idlc')
-rw-r--r--idlc/source/astexpression.cxx2
1 files changed, 1 insertions, 1 deletions
diff --git a/idlc/source/astexpression.cxx b/idlc/source/astexpression.cxx
index c1563a3dfb14..5da3616e1c00 100644
--- a/idlc/source/astexpression.cxx
+++ b/idlc/source/astexpression.cxx
@@ -1083,7 +1083,7 @@ const sal_Char* SAL_CALL exprTypeToString(ExprType t)
return "none";
}
- return ("unknown");
+ return "unknown";
}
/* vim:set shiftwidth=4 softtabstop=4 expandtab: */