From 084f454e8caa2d9e43f7bdea098538bfb87423d8 Mon Sep 17 00:00:00 2001 From: Stephan Bergmann Date: Thu, 26 Oct 2017 10:27:48 +0200 Subject: More loplugin:unnecessaryparen 81892b2037453108b9bde1512a500cf3b2ce438a "loplugin:unnecessaryparen when compiling as C++17, so the ParenExpr is no longer hidden behind ExprWithCleanups/CXXConstructExpr/MaterializedTemporaryExpr wrappers" gave me the idea to generally look though IgnoreImplicit instead of IngoreImpCasts in loplugin:unnecessaryparen. However, that would still not look through implicit CXXConstructExpr, so would still not have found the occurrences in 81892b2037453108b9bde1512a500cf3b2ce438a when compiling in pre-C++17 mode. Therefore, let ignoreAllImplicit also look through CXXConstructExpr. (I am not entirely sure in which situations non-implicit CXXConstructExpr---that should thus not be ignored---would occur, but assume they would be underneath something like a CXXFunctionalCastExpr, which is not ignored.) Change-Id: I947d08742e1809150ecc34a7abe84cca5e0ce843 --- lotuswordpro/source/filter/lwpfont.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'lotuswordpro') diff --git a/lotuswordpro/source/filter/lwpfont.cxx b/lotuswordpro/source/filter/lwpfont.cxx index a192a8ed55e0..e9011a5ea3db 100644 --- a/lotuswordpro/source/filter/lwpfont.cxx +++ b/lotuswordpro/source/filter/lwpfont.cxx @@ -231,7 +231,7 @@ void LwpFontTableEntry::Read(LwpObjectStream *pStrm) OUString LwpFontTableEntry::GetFaceName() { - return (m_WindowsFaceName.str()); + return m_WindowsFaceName.str(); } void LwpFontTableEntry::RegisterFontDecl() -- cgit