From 91b4e4531621b7afb2dbab1a8aa62c92da66951a Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Sun, 21 Jan 2018 13:07:10 +0200 Subject: new loplugin: pointerbool look for possibly bogus implicit conversions to bool when passing (normally pointer) args to bool params. this plugin comes in the wake of a couple of bugs caused by refactoring, where some of the call sites were not currently updated. Of the changes, the following are real bugs: desktop/../dp_persmap.cxx StartInputFieldDlg in sw/../fldmgr.cxx which occurred as a result of commit 39d719a80d8c87856c84e3ecd569d45fa6f8a30e Date: Tue May 3 11:39:37 2016 +0200 tdf#99529 sw: don't pop up input field dialog before inserting field CSerializationURLEncoded::encode_and_append in forms/../serialization_urlencoded.cxx XclExpCFImpl::XclExpCFImpl in sc/../xecontent.cxx I have no idea how to properly fix this, just made a guess. SwDocTest::test64kPageDescs in sw/qa/core/uwriter.cxx which looks like a simple copy/paste error. Change-Id: I795ebd5ef485a1d36863dc27fe13832989f5a441 Reviewed-on: https://gerrit.libreoffice.org/48291 Reviewed-by: Noel Grandin Tested-by: Noel Grandin --- desktop/source/deployment/dp_persmap.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'desktop/source/deployment') diff --git a/desktop/source/deployment/dp_persmap.cxx b/desktop/source/deployment/dp_persmap.cxx index e254d9b67c0a..2e5a047cd864 100644 --- a/desktop/source/deployment/dp_persmap.cxx +++ b/desktop/source/deployment/dp_persmap.cxx @@ -91,7 +91,7 @@ static OString encodeString( const OString& rStr) c += (c <= 0x09) ? '0' : 'A'-10; } else if( c == '%') aEncStr.append( '%'); - aEncStr.append( c); + aEncStr.append( char(c) ); } return aEncStr.makeStringAndClear(); -- cgit