bool improvements

Change-Id: Id1a0df17a28372f9d7c3a88424cf7656fed46f9a

svtools/source/config/menuoptions.cxx

@@ -237,7 +237,7 @@ SvtMenuOptions_Impl::SvtMenuOptions_Impl()
     {
         // Safe impossible cases.
         // Check any for valid value.
-        DBG_ASSERT( !(seqValues[nProperty].hasValue()==sal_False), "SvtMenuOptions_Impl::SvtMenuOptions_Impl()\nInvalid property value for property detected!\n" );
+        DBG_ASSERT( seqValues[nProperty].hasValue(), "SvtMenuOptions_Impl::SvtMenuOptions_Impl()\nInvalid property value for property detected!\n" );
 
         if (!seqValues[nProperty].hasValue())
             continue;

svtools/source/config/miscopt.cxx

@@ -289,7 +289,7 @@ SvtMiscOptions_Impl::SvtMiscOptions_Impl()
     sal_Int32 nPropertyCount = seqValues.getLength();
     for( sal_Int32 nProperty=0; nProperty<nPropertyCount; ++nProperty )
     {
-        if (seqValues[nProperty].hasValue()==sal_False)
+        if (!seqValues[nProperty].hasValue())
             continue;
         switch( nProperty )
         {
@@ -435,7 +435,7 @@ void SvtMiscOptions_Impl::Load( const Sequence< OUString >& rPropertyNames )
     sal_Int32 nPropertyCount = seqValues.getLength();
     for( sal_Int32 nProperty=0; nProperty<nPropertyCount; ++nProperty )
     {
-        if (seqValues[nProperty].hasValue()==sal_False)
+        if (!seqValues[nProperty].hasValue())
             continue;
         switch( lcl_MapPropertyName(rPropertyNames[nProperty], aInternalPropertyNames) )
         {

svtools/source/config/optionsdrawinglayer.cxx

@@ -339,7 +339,7 @@ SvtOptionsDrawinglayer_Impl::SvtOptionsDrawinglayer_Impl() :
     sal_Int32 nPropertyCount = seqValues.getLength();
     for(sal_Int32 nProperty=0; nProperty<nPropertyCount; ++nProperty )
     {
-        if (seqValues[nProperty].hasValue()==sal_False)
+        if (!seqValues[nProperty].hasValue())
             continue;
 
         switch( nProperty )

svtools/source/config/slidesorterbaropt.cxx

@@ -129,7 +129,7 @@ SvtSlideSorterBarOptions_Impl::SvtSlideSorterBarOptions_Impl()
     // Copy values from list in right order to our internal member.
     for( sal_Int32 nProperty=0; nProperty<seqValues.getLength(); ++nProperty )
     {
-        if (seqValues[nProperty].hasValue()==sal_False)
+        if (!seqValues[nProperty].hasValue())
             continue;
         switch( nProperty )
         {
@@ -207,7 +207,7 @@ void SvtSlideSorterBarOptions_Impl::Load( const Sequence< OUString >& rPropertyN
     // Copy values from list in right order to our internal member.
     for( sal_Int32 nProperty=0; nProperty<seqValues.getLength(); ++nProperty )
     {
-        if (seqValues[nProperty].hasValue()==sal_False)
+        if (!seqValues[nProperty].hasValue())
             continue;
         switch( lcl_MapPropertyName(rPropertyNames[nProperty], aInternalPropertyNames) )
         {

svtools/source/config/toolpanelopt.cxx

@@ -133,7 +133,7 @@ SvtToolPanelOptions_Impl::SvtToolPanelOptions_Impl()
     // Copy values from list in right order to our internal member.
     for( sal_Int32 nProperty=0; nProperty<seqValues.getLength(); ++nProperty )
     {
-        if (seqValues[nProperty].hasValue()==sal_False)
+        if (!seqValues[nProperty].hasValue())
             continue;
         switch( nProperty )
         {
@@ -205,7 +205,7 @@ void SvtToolPanelOptions_Impl::Load( const Sequence< OUString >& rPropertyNames
     // Copy values from list in right order to our internal member.
     for( sal_Int32 nProperty=0; nProperty<seqValues.getLength(); ++nProperty )
     {
-        if (seqValues[nProperty].hasValue()==sal_False)
+        if (!seqValues[nProperty].hasValue())
             continue;
         switch( lcl_MapPropertyName(rPropertyNames[nProperty], aInternalPropertyNames) )
         {

svtools/source/hatchwindow/hatchwindow.cxx

@@ -116,7 +116,7 @@ uno::Any SAL_CALL VCLXHatchWindow::queryInterface( const uno::Type & rType )
     uno::Any aReturn( ::cppu::queryInterface( rType,
                                            static_cast< embed::XHatchWindow* >( this ) ) );
 
-    if ( aReturn.hasValue() == sal_True )
+    if ( aReturn.hasValue() )
     {
         return aReturn ;
     }

svtools/source/misc/xwindowitem.cxx

@@ -59,7 +59,7 @@ bool XWindowItem::operator == ( const SfxPoolItem & rAttr ) const
     DBG_ASSERT( SfxPoolItem::operator==(rAttr), "unequal types" );
 
     const XWindowItem * pItem = dynamic_cast< const XWindowItem * >(&rAttr);
-    return pItem ? m_xWin == pItem->m_xWin : 0;
+    return pItem && m_xWin == pItem->m_xWin;
 }