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

Arnaud Versini Arnaud.Versini at libreoffice.org
Sun May 14 11:33:45 UTC 2017


 accessibility/source/extended/accessibletabbarpage.cxx        |    8 -----
 accessibility/source/extended/accessibletabbarpagelist.cxx    |   14 ----------
 accessibility/source/standard/vclxaccessiblemenuitem.cxx      |    8 -----
 accessibility/source/standard/vclxaccessiblemenuseparator.cxx |    2 -
 4 files changed, 32 deletions(-)

New commits:
commit d17c7a7ec842657dcfb29fdbb222b09f0c879944
Author: Arnaud Versini <Arnaud.Versini at libreoffice.org>
Date:   Sun May 7 13:41:01 2017 +0200

    accessibility: Remove useless locks.
    
    Change-Id: I61bb14e33db1e32f0eb4c27cbb62e7f37006213f
    Reviewed-on: https://gerrit.libreoffice.org/37342
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>

diff --git a/accessibility/source/extended/accessibletabbarpage.cxx b/accessibility/source/extended/accessibletabbarpage.cxx
index 3f185a678754..9cfad8b674bd 100644
--- a/accessibility/source/extended/accessibletabbarpage.cxx
+++ b/accessibility/source/extended/accessibletabbarpage.cxx
@@ -251,8 +251,6 @@ namespace accessibility
 
     sal_Int32 AccessibleTabBarPage::getAccessibleChildCount()
     {
-        OExternalLockGuard aGuard( this );
-
         return 0;
     }
 
@@ -290,8 +288,6 @@ namespace accessibility
 
     sal_Int16 AccessibleTabBarPage::getAccessibleRole(  )
     {
-        OExternalLockGuard aGuard( this );
-
         return AccessibleRole::PAGE_TAB;
     }
 
@@ -359,8 +355,6 @@ namespace accessibility
 
     Reference< XAccessible > AccessibleTabBarPage::getAccessibleAtPoint( const awt::Point& )
     {
-        OExternalLockGuard aGuard( this );
-
         return Reference< XAccessible >();
     }
 
@@ -435,8 +429,6 @@ namespace accessibility
 
     OUString AccessibleTabBarPage::getToolTipText(  )
     {
-        OExternalLockGuard aGuard( this );
-
         return OUString();
     }
 
diff --git a/accessibility/source/extended/accessibletabbarpagelist.cxx b/accessibility/source/extended/accessibletabbarpagelist.cxx
index 8853ac58a8b7..1bb3b73ddd7f 100644
--- a/accessibility/source/extended/accessibletabbarpagelist.cxx
+++ b/accessibility/source/extended/accessibletabbarpagelist.cxx
@@ -452,24 +452,18 @@ namespace accessibility
 
     sal_Int16 AccessibleTabBarPageList::getAccessibleRole(  )
     {
-        OExternalLockGuard aGuard( this );
-
         return AccessibleRole::PAGE_TAB_LIST;
     }
 
 
     OUString AccessibleTabBarPageList::getAccessibleDescription( )
     {
-        OExternalLockGuard aGuard( this );
-
         return OUString();
     }
 
 
     OUString AccessibleTabBarPageList::getAccessibleName(  )
     {
-        OExternalLockGuard aGuard( this );
-
         return OUString();
     }
 
@@ -605,16 +599,12 @@ namespace accessibility
 
     OUString AccessibleTabBarPageList::getTitledBorderText(  )
     {
-        OExternalLockGuard aGuard( this );
-
         return OUString();
     }
 
 
     OUString AccessibleTabBarPageList::getToolTipText(  )
     {
-        OExternalLockGuard aGuard( this );
-
         return OUString();
     }
 
@@ -662,16 +652,12 @@ namespace accessibility
 
     void AccessibleTabBarPageList::selectAllAccessibleChildren(  )
     {
-        OExternalLockGuard aGuard( this );
-
         selectAccessibleChild( 0 );
     }
 
 
     sal_Int32 AccessibleTabBarPageList::getSelectedAccessibleChildCount(  )
     {
-        OExternalLockGuard aGuard( this );
-
         return 1;
     }
 
diff --git a/accessibility/source/standard/vclxaccessiblemenuitem.cxx b/accessibility/source/standard/vclxaccessiblemenuitem.cxx
index 9db5a5357b00..7c6001721dfb 100644
--- a/accessibility/source/standard/vclxaccessiblemenuitem.cxx
+++ b/accessibility/source/standard/vclxaccessiblemenuitem.cxx
@@ -194,8 +194,6 @@ sal_Int16 VCLXAccessibleMenuItem::getAccessibleRole(  )
 
 sal_Int32 VCLXAccessibleMenuItem::getCaretPosition()
 {
-    OExternalLockGuard aGuard( this );
-
     return -1;
 }
 
@@ -399,8 +397,6 @@ sal_Bool VCLXAccessibleMenuItem::copyText( sal_Int32 nStartIndex, sal_Int32 nEnd
 
 sal_Int32 VCLXAccessibleMenuItem::getAccessibleActionCount( )
 {
-    OExternalLockGuard aGuard( this );
-
     return 1;
 }
 
@@ -547,8 +543,6 @@ sal_Bool VCLXAccessibleMenuItem::setCurrentValue( const Any& aNumber )
 
 Any VCLXAccessibleMenuItem::getMaximumValue(  )
 {
-    OExternalLockGuard aGuard( this );
-
     Any aValue;
     aValue <<= (sal_Int32) 1;
 
@@ -558,8 +552,6 @@ Any VCLXAccessibleMenuItem::getMaximumValue(  )
 
 Any VCLXAccessibleMenuItem::getMinimumValue(  )
 {
-    OExternalLockGuard aGuard( this );
-
     Any aValue;
     aValue <<= (sal_Int32) 0;
 
diff --git a/accessibility/source/standard/vclxaccessiblemenuseparator.cxx b/accessibility/source/standard/vclxaccessiblemenuseparator.cxx
index ca7162de520f..94798416dba8 100644
--- a/accessibility/source/standard/vclxaccessiblemenuseparator.cxx
+++ b/accessibility/source/standard/vclxaccessiblemenuseparator.cxx
@@ -62,8 +62,6 @@ Sequence< OUString > VCLXAccessibleMenuSeparator::getSupportedServiceNames()
 
 sal_Int16 VCLXAccessibleMenuSeparator::getAccessibleRole(  )
 {
-    OExternalLockGuard aGuard( this );
-
     return AccessibleRole::SEPARATOR;
 }
 


More information about the Libreoffice-commits mailing list