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 --- toolkit/source/controls/unocontrols.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'toolkit') diff --git a/toolkit/source/controls/unocontrols.cxx b/toolkit/source/controls/unocontrols.cxx index dd2d11eaccd2..98b4b8c0b52a 100644 --- a/toolkit/source/controls/unocontrols.cxx +++ b/toolkit/source/controls/unocontrols.cxx @@ -3638,7 +3638,7 @@ void UnoDateFieldControl::createPeer( const uno::Reference< awt::XToolkit > & rx xField->setFirst( mnFirst ); xField->setLast( mnLast ); if ( mbLongFormat != TRISTATE_INDET ) - xField->setLongFormat( mbLongFormat ); + xField->setLongFormat( mbLongFormat != TRISTATE_FALSE); } -- cgit