[ooo-build-commit] .: accessibility/source

Noel Power noelp at kemper.freedesktop.org
Fri Oct 1 04:24:06 PDT 2010


 accessibility/source/standard/vclxaccessiblebox.cxx         |    2 -
 accessibility/source/standard/vclxaccessiblelist.cxx        |    7 ------
 accessibility/source/standard/vclxaccessibletoolboxitem.cxx |   14 ------------
 3 files changed, 3 insertions(+), 20 deletions(-)

New commits:
commit 8d890248dc4e7ca5c5927ba938cd0182b0a9f45b
Author: Jacopo Nespolo <j.nespolo at gmail.com>
Date:   Fri Oct 1 12:23:16 2010 +0100

    Removing dead code and cleaning up some comments

diff --git a/accessibility/source/standard/vclxaccessiblebox.cxx b/accessibility/source/standard/vclxaccessiblebox.cxx
index de6eac8..25e66a0 100644
--- a/accessibility/source/standard/vclxaccessiblebox.cxx
+++ b/accessibility/source/standard/vclxaccessiblebox.cxx
@@ -149,7 +149,7 @@ void VCLXAccessibleBox::ProcessWindowEvent (const VclWindowEvent& rVclWindowEven
         {
             // Selection is handled by VCLXAccessibleList which operates on
             // the same VCL object as this box does.  In case of the
-            // combobox, however, we have to help the list with providing
+            // combobox, however, we have to help by providing the list with
             // the text of the currently selected item.
             VCLXAccessibleList* pList = static_cast<VCLXAccessibleList*>(m_xList.get());
             if (pList != NULL && m_xText.is())
diff --git a/accessibility/source/standard/vclxaccessiblelist.cxx b/accessibility/source/standard/vclxaccessiblelist.cxx
index 68fd2b5..0e10b31 100644
--- a/accessibility/source/standard/vclxaccessiblelist.cxx
+++ b/accessibility/source/standard/vclxaccessiblelist.cxx
@@ -71,7 +71,7 @@ VCLXAccessibleList::VCLXAccessibleList (VCLXWindow* pVCLWindow, BoxType aBoxType
       m_bVisible				( true ),
       m_xParent                 ( _xParent )
 {
-    // Because combo boxes and list boxes have the no common interface for
+    // Because combo boxes and list boxes don't have a common interface for
     // methods with identical signature we have to write down twice the
     // same code.
     switch (m_aBoxType)
@@ -125,10 +125,6 @@ void SAL_CALL VCLXAccessibleList::disposing (void)
 
 void VCLXAccessibleList::clearItems()
 {
-//	ListItems::iterator aEnd = m_aAccessibleChildren.end();
-//	for (ListItems::iterator aIter = m_aAccessibleChildren.begin(); aIter != aEnd; ++aIter)
-//		::comphelper::disposeComponent(*aIter);
-
     // Clear the list itself and delete all the rest.
     ListItems().swap(m_aAccessibleChildren); // clear and minimize
 }
@@ -790,7 +786,6 @@ void SAL_CALL VCLXAccessibleList::deselectAccessibleChild( sal_Int32 nSelectedCh
         UpdateSelection_Impl();
 }
 // -----------------------------------------------------------------------------
-// accessibility::XAccessibleComponent
 awt::Rectangle VCLXAccessibleList::implGetBounds() throw (uno::RuntimeException)
 {
     awt::Rectangle aBounds ( 0, 0, 0, 0 );
diff --git a/accessibility/source/standard/vclxaccessibletoolboxitem.cxx b/accessibility/source/standard/vclxaccessibletoolboxitem.cxx
index 6f9f51b..0aac393 100644
--- a/accessibility/source/standard/vclxaccessibletoolboxitem.cxx
+++ b/accessibility/source/standard/vclxaccessibletoolboxitem.cxx
@@ -137,17 +137,6 @@ VCLXAccessibleToolBoxItem::~VCLXAccessibleToolBoxItem()
     if ( m_pToolBox && m_nItemId > 0 && ( _bAsName || m_pToolBox->GetButtonType() != BUTTON_SYMBOL ) )
     {
         sRet = m_pToolBox->GetItemText( m_nItemId );
-//OJ #108243# we only read the name of the toolboxitem
-//
-//		Window* pItemWindow = m_pToolBox->GetItemWindow( m_nItemId );
-//		if ( pItemWindow && pItemWindow->GetAccessible().is() &&
-//			 pItemWindow->GetAccessible()->getAccessibleContext().is() )
-//		{
-//			::rtl::OUString sWinText = pItemWindow->GetAccessible()->getAccessibleContext()->getAccessibleName();
-//			if ( ( sRet.getLength() > 0 ) && ( sWinText.getLength() > 0 ) )
-//				sRet += String( RTL_CONSTASCII_USTRINGPARAM( " " ) );
-//			sRet += sWinText;
-//		}
     }
     return sRet;
 }
@@ -270,11 +259,10 @@ void VCLXAccessibleToolBoxItem::implGetSelection( sal_Int32& nStartIndex, sal_In
 IMPLEMENT_FORWARD_REFCOUNT( VCLXAccessibleToolBoxItem, AccessibleTextHelper_BASE )
 Any SAL_CALL VCLXAccessibleToolBoxItem::queryInterface( const Type& _rType ) throw (RuntimeException)
 {
-    // --> PB 2004-09-03 #i33611# - toolbox buttons without text don't support XAccessibleText
+    // #i33611# - toolbox buttons without text don't support XAccessibleText
     if ( _rType == ::getCppuType( ( const Reference< XAccessibleText >* ) 0 )
         && ( !m_pToolBox || m_pToolBox->GetButtonType() == BUTTON_SYMBOL ) )
         return Any();
-    // <--
 
     ::com::sun::star::uno::Any aReturn = AccessibleTextHelper_BASE::queryInterface( _rType );
     if ( !aReturn.hasValue() )


More information about the ooo-build-commit mailing list