Date: prev next · Thread: first prev next last
2011 Archives by date, by thread · List index


Please, someone can review this? Thkz.

revol_
diff --git a/sw/source/core/access/acccell.cxx b/sw/source/core/access/acccell.cxx
index dbd1b51..2479c5c 100644
--- a/sw/source/core/access/acccell.cxx
+++ b/sw/source/core/access/acccell.cxx
@@ -121,7 +121,7 @@ SwAccessibleCell::SwAccessibleCell( SwAccessibleMap *pInitMap,
 sal_Bool SwAccessibleCell::_InvalidateMyCursorPos()
 {
     sal_Bool bNew = IsSelected();
-    sal_Bool bOld;
+    sal_Bool bOld = sal_False;
     {
         osl::MutexGuard aGuard( aMutex );
         bOld = bIsSelected;
@@ -349,7 +349,7 @@ sal_Bool SwAccessibleCell::setCurrentValue( const uno::Any& aNumber )
     SolarMutexGuard aGuard;
     CHECK_FOR_DEFUNC( XAccessibleValue );
 
-    double fValue = 0;
+    const double fValue = 0;
     sal_Bool bValid = (aNumber >>= fValue);
     if( bValid )
     {
diff --git a/sw/source/core/access/acccell.cxx b/sw/source/core/access/acccell.cxx
index dbd1b51..2479c5c 100644
--- a/sw/source/core/access/acccell.cxx
+++ b/sw/source/core/access/acccell.cxx
@@ -121,7 +121,7 @@ SwAccessibleCell::SwAccessibleCell( SwAccessibleMap *pInitMap,
 sal_Bool SwAccessibleCell::_InvalidateMyCursorPos()
 {
     sal_Bool bNew = IsSelected();
-    sal_Bool bOld;
+    sal_Bool bOld = sal_False;
     {
         osl::MutexGuard aGuard( aMutex );
         bOld = bIsSelected;
@@ -349,7 +349,7 @@ sal_Bool SwAccessibleCell::setCurrentValue( const uno::Any& aNumber )
     SolarMutexGuard aGuard;
     CHECK_FOR_DEFUNC( XAccessibleValue );
 
-    double fValue = 0;
+    const double fValue = 0;
     sal_Bool bValid = (aNumber >>= fValue);
     if( bValid )
     {
diff --git a/sw/source/core/access/acccontext.cxx b/sw/source/core/access/acccontext.cxx
index 127820f..85a55cd 100644
--- a/sw/source/core/access/acccontext.cxx
+++ b/sw/source/core/access/acccontext.cxx
@@ -507,7 +507,7 @@ void SwAccessibleContext::FireVisibleDataEvent()
 }
 
 void SwAccessibleContext::FireStateChangedEvent( sal_Int16 nState,
-                                                 sal_Bool bNewState )
+                                                 const sal_Bool bNewState )
 {
     AccessibleEventObject aEvent;
 
@@ -550,7 +550,7 @@ void SwAccessibleContext::GetStates(
 
 sal_Bool SwAccessibleContext::IsEditableState()
 {
-    sal_Bool bRet;
+    sal_Bool bRet = sal_False;
     {
         osl::MutexGuard aGuard( aMutex );
         bRet = bIsEditableState;
@@ -601,7 +601,7 @@ sal_Int32 SAL_CALL SwAccessibleContext::getAccessibleChildCount( void )
 }
 
 uno::Reference< XAccessible> SAL_CALL
-    SwAccessibleContext::getAccessibleChild( sal_Int32 nIndex )
+    SwAccessibleContext::getAccessibleChild( const sal_Int32 nIndex )
         throw (uno::RuntimeException, lang::IndexOutOfBoundsException)
 {
     SolarMutexGuard aGuard;
diff --git a/sw/source/core/access/acccell.cxx b/sw/source/core/access/acccell.cxx
index dbd1b51..2479c5c 100644
--- a/sw/source/core/access/acccell.cxx
+++ b/sw/source/core/access/acccell.cxx
@@ -121,7 +121,7 @@ SwAccessibleCell::SwAccessibleCell( SwAccessibleMap *pInitMap,
 sal_Bool SwAccessibleCell::_InvalidateMyCursorPos()
 {
     sal_Bool bNew = IsSelected();
-    sal_Bool bOld;
+    sal_Bool bOld = sal_False;
     {
         osl::MutexGuard aGuard( aMutex );
         bOld = bIsSelected;
@@ -349,7 +349,7 @@ sal_Bool SwAccessibleCell::setCurrentValue( const uno::Any& aNumber )
     SolarMutexGuard aGuard;
     CHECK_FOR_DEFUNC( XAccessibleValue );
 
-    double fValue = 0;
+    const double fValue = 0;
     sal_Bool bValid = (aNumber >>= fValue);
     if( bValid )
     {
diff --git a/sw/source/core/access/acccontext.cxx b/sw/source/core/access/acccontext.cxx
index 127820f..978dcad 100644
--- a/sw/source/core/access/acccontext.cxx
+++ b/sw/source/core/access/acccontext.cxx
@@ -507,7 +507,7 @@ void SwAccessibleContext::FireVisibleDataEvent()
 }
 
 void SwAccessibleContext::FireStateChangedEvent( sal_Int16 nState,
-                                                 sal_Bool bNewState )
+                                                 const sal_Bool bNewState )
 {
     AccessibleEventObject aEvent;
 
@@ -550,7 +550,7 @@ void SwAccessibleContext::GetStates(
 
 sal_Bool SwAccessibleContext::IsEditableState()
 {
-    sal_Bool bRet;
+    sal_Bool bRet = sal_False;
     {
         osl::MutexGuard aGuard( aMutex );
         bRet = bIsEditableState;
@@ -601,7 +601,7 @@ sal_Int32 SAL_CALL SwAccessibleContext::getAccessibleChildCount( void )
 }
 
 uno::Reference< XAccessible> SAL_CALL
-    SwAccessibleContext::getAccessibleChild( sal_Int32 nIndex )
+    SwAccessibleContext::getAccessibleChild( const sal_Int32 nIndex )
         throw (uno::RuntimeException, lang::IndexOutOfBoundsException)
 {
     SolarMutexGuard aGuard;
@@ -771,7 +771,7 @@ void SAL_CALL SwAccessibleContext::removeEventListener(
     if (xListener.is())
     {
         SolarMutexGuard aGuard;
-        sal_Int32 nListenerCount = comphelper::AccessibleEventNotifier::removeEventListener( 
nClientId, xListener );
+        const sal_Int32 nListenerCount = comphelper::AccessibleEventNotifier::removeEventListener( 
nClientId, xListener );
         if ( !nListenerCount )
         {
             // no listeners anymore
@@ -1130,7 +1130,7 @@ void SwAccessibleContext::Dispose( sal_Bool bRecursive )
 }
 
 void SwAccessibleContext::DisposeChild( const SwAccessibleChild& rChildFrmOrObj,
-                                        sal_Bool bRecursive )
+                                        const sal_Bool bRecursive )
 {
     SolarMutexGuard aGuard;
 
@@ -1176,7 +1176,7 @@ void SwAccessibleContext::InvalidatePosOrSize( const SwRect& )
 
     OSL_ENSURE( GetFrm() && !GetFrm()->Frm().IsEmpty(), "context should have a size" );
 
-    sal_Bool bIsOldShowingState;
+    sal_Bool bIsOldShowingState = sal_False;
     sal_Bool bIsNewShowingState = IsShowing( *(GetMap()) );
     {
         osl::MutexGuard aShowingStateGuard( aMutex );
@@ -1333,7 +1333,7 @@ void SwAccessibleContext::InvalidateStates( tAccessibleStates _nStates )
         {
             if( (_nStates & ACC_STATE_EDITABLE) != 0 )
             {
-                sal_Bool bIsOldEditableState;
+                sal_Bool bIsOldEditableState = sal_False;
                 sal_Bool bIsNewEditableState = IsEditable( pVSh );
                 {
                     osl::MutexGuard aGuard( aMutex );
@@ -1347,7 +1347,7 @@ void SwAccessibleContext::InvalidateStates( tAccessibleStates _nStates )
             }
             if( (_nStates & ACC_STATE_OPAQUE) != 0 )
             {
-                sal_Bool bIsOldOpaqueState;
+                sal_Bool bIsOldOpaqueState = sal_False;
                 sal_Bool bIsNewOpaqueState = IsOpaque( pVSh );
                 {
                     osl::MutexGuard aGuard( aMutex );
@@ -1366,7 +1366,7 @@ void SwAccessibleContext::InvalidateStates( tAccessibleStates _nStates )
 }
 // <--
 
-void SwAccessibleContext::InvalidateRelation( sal_uInt16 nType )
+void SwAccessibleContext::InvalidateRelation( const sal_uInt16 nType )
 {
     AccessibleEventObject aEvent;
     aEvent.EventId = nType;
@@ -1407,8 +1407,8 @@ sal_Bool SwAccessibleContext::HasCursor()
     return sal_False;
 }
 
-sal_Bool SwAccessibleContext::Select( SwPaM *pPaM, SdrObject *pObj,
-                                      sal_Bool bAdd )
+sal_Bool SwAccessibleContext::Select( const SwPaM *pPaM, const SdrObject *pObj,
+                                     const sal_Bool bAdd )
 {
     SwCrsrShell* pCrsrShell = GetCrsrShell();
     if( !pCrsrShell )
@@ -1454,7 +1454,7 @@ sal_Bool SwAccessibleContext::Select( SwPaM *pPaM, SdrObject *pObj,
     return bRet;
 }
 
-OUString SwAccessibleContext::GetResource( sal_uInt16 nResId,
+OUString SwAccessibleContext::GetResource( const sal_uInt16 nResId,
                                            const OUString *pArg1,
                                            const OUString *pArg2 )
 {

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.