[Libreoffice-commits] core.git: sw/source

Tor Lillqvist tml at collabora.com
Thu Oct 30 23:47:47 PDT 2014


 sw/source/uibase/uno/SwXDocumentSettings.cxx |    5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

New commits:
commit 4482f50197f8787f8e187def2cc96758e3c31800
Author: Tor Lillqvist <tml at collabora.com>
Date:   Fri Oct 31 08:32:31 2014 +0200

    WaE: VarDecl, use "bool" instead of "sal_Bool" [loplugin:salbool]
    
    This likely is a false warning or over-enthusiasm in the salbool plug-in, but
    on the other hand, I don't understand why the recently added case needs to use
    different coding idioms from the preceding ones in the function? So make the
    code more uniform.
    
    Change-Id: I518022c59f6bd54740676cba95670e78b9a40d19

diff --git a/sw/source/uibase/uno/SwXDocumentSettings.cxx b/sw/source/uibase/uno/SwXDocumentSettings.cxx
index ec71f6d..b34dd65 100644
--- a/sw/source/uibase/uno/SwXDocumentSettings.cxx
+++ b/sw/source/uibase/uno/SwXDocumentSettings.cxx
@@ -1248,9 +1248,8 @@ void SwXDocumentSettings::_getSingleValue( const comphelper::PropertyInfo & rInf
         break;
         case HANDLE_PROP_LINE_SPACING_SHRINKS_FIRST_LINE:
         {
-            sal_Bool const bTmp(mpDoc->getIDocumentSettingAccess().get(
-                IDocumentSettingAccess::PROP_LINE_SPACING_SHRINKS_FIRST_LINE));
-            rValue <<= bTmp;
+            sal_Bool bTmp = mpDoc->getIDocumentSettingAccess().get( IDocumentSettingAccess::PROP_LINE_SPACING_SHRINKS_FIRST_LINE);
+            rValue.setValue( &bTmp, ::getBooleanCppuType() );
         }
         break;
         default:


More information about the Libreoffice-commits mailing list