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

sonakshi nathani sonakshinathani at gmail.com
Wed Jun 26 07:12:05 PDT 2013


 svx/source/accessibility/AccessibleControlShape.cxx |   47 --------------------
 1 file changed, 2 insertions(+), 45 deletions(-)

New commits:
commit 51c79f97a90eaf469d1d2531f3f0eccfafe0f94c
Author: sonakshi nathani <sonakshinathani at gmail.com>
Date:   Wed Jun 26 17:59:29 2013 +0530

    "fdo#62475 removed pointless comments
    
    All the comments of the form //-------- are removed from /svx/source/accessibilty/AccessibleControlShape.cxx"
    
    Change-Id: I7ae9f656939e2512ce503a5ba103d765594f20ac
    Reviewed-on: https://gerrit.libreoffice.org/4543
    Reviewed-by: Miklos Vajna <vmiklos at suse.cz>
    Tested-by: Miklos Vajna <vmiklos at suse.cz>

diff --git a/svx/source/accessibility/AccessibleControlShape.cxx b/svx/source/accessibility/AccessibleControlShape.cxx
index 0f56193..ebbd599 100644
--- a/svx/source/accessibility/AccessibleControlShape.cxx
+++ b/svx/source/accessibility/AccessibleControlShape.cxx
@@ -50,28 +50,23 @@ using namespace ::com::sun::star::reflection;
 using namespace ::com::sun::star::drawing;
 using namespace ::com::sun::star::container;
 
-//--------------------------------------------------------------------
 namespace
 {
-    //................................................................
     const OUString& lcl_getNamePropertyName( )
     {
         static OUString s_sNamePropertyName( "Name" );
         return s_sNamePropertyName;
     }
-    //................................................................
     const OUString& lcl_getDescPropertyName( )
     {
         static OUString s_sDescPropertyDesc( "HelpText" );
         return s_sDescPropertyDesc;
     }
-    //................................................................
     const OUString& lcl_getLabelPropertyName( )
     {
         static OUString s_sLabelPropertyLabel( "Label" );
         return s_sLabelPropertyLabel;
     }
-    //................................................................
     // return the property which should be used as AccessibleName
     const OUString& lcl_getPreferredAccNameProperty( const Reference< XPropertySetInfo >& _rxPSI )
     {
@@ -81,7 +76,6 @@ namespace
             return lcl_getNamePropertyName();
     }
 
-    //................................................................
     // determines whether or not a state which belongs to the inner context needs to be forwarded to the "composed"
     // context
     bool    isComposedState( const sal_Int16 _nState )
@@ -97,9 +91,7 @@ namespace
                 );
     }
 
-    //................................................................
-    /** determines whether the given control is in alive mode
-    */
+    /// determines whether the given control is in alive mode
     inline  bool    isAliveMode( const Reference< XControl >& _rxControl )
     {
         OSL_PRECOND( _rxControl.is(), "AccessibleControlShape::isAliveMode: invalid control" );
@@ -107,11 +99,7 @@ namespace
     }
 }
 
-//=============================================================================
 //= AccessibleControlShape
-//=============================================================================
-
-//-----------------------------------------------------------------------------
 AccessibleControlShape::AccessibleControlShape (
     const AccessibleShapeInfo& rShapeInfo,
     const AccessibleShapeTreeInfo& rShapeTreeInfo)
@@ -132,7 +120,6 @@ AccessibleControlShape::AccessibleControlShape (
     osl_atomic_decrement( &m_refCount );
 }
 
-//-----------------------------------------------------------------------------
 AccessibleControlShape::~AccessibleControlShape (void)
 {
     m_pChildManager->release();
@@ -147,7 +134,6 @@ AccessibleControlShape::~AccessibleControlShape (void)
         // ourself) to this proxy, and thus delete it
 }
 
-//-----------------------------------------------------------------------------
 SdrObject* AccessibleControlShape::getSdrObject() const
 {
     return GetSdrObjectFromXShape (mxShape);
@@ -166,7 +152,6 @@ namespace {
     }
 }
 
-//-----------------------------------------------------------------------------
 void AccessibleControlShape::Init()
 {
     AccessibleShape::Init();
@@ -205,7 +190,6 @@ void AccessibleControlShape::Init()
 
         if ( pView && pViewWindow && pUnoObjectImpl )
         {
-            // .................................................................
             // get the context of the control - it will be our "inner" context
             m_xUnoControl = pUnoObjectImpl->GetUnoControl( *pView, *pViewWindow );
 
@@ -237,7 +221,6 @@ void AccessibleControlShape::Init()
                 OSL_ENSURE( xNativeControlContext.is(), "AccessibleControlShape::Init: no AccessibleContext for the control!" );
                 m_aControlContext = WeakReference< XAccessibleContext >( xNativeControlContext );
 
-                // .................................................................
                 // add as listener to the context - we want to multiplex some states
                 if ( isAliveMode( m_xUnoControl ) && xNativeControlContext.is() )
                 {   // (but only in alive mode)
@@ -256,7 +239,6 @@ void AccessibleControlShape::Init()
                     m_pChildManager->setTransientChildren( !xStates.is() || xStates->contains( AccessibleStateType::MANAGES_DESCENDANTS ) );
                 }
 
-                // .................................................................
                 // finally, aggregate a proxy for the control context
                 // first a factory for the proxy
                 Reference< XProxyFactory > xFactory = ProxyFactory::create( comphelper::getProcessComponentContext() );
@@ -292,14 +274,11 @@ void AccessibleControlShape::Init()
     }
 }
 
-//-----------------------------------------------------------------------------
 Reference< XAccessibleContext > SAL_CALL AccessibleControlShape::getAccessibleContext(void) throw (RuntimeException)
 {
     return AccessibleShape::getAccessibleContext ();
 }
 
-
-//-----------------------------------------------------------------------------
 void SAL_CALL AccessibleControlShape::grabFocus(void)  throw (RuntimeException)
 {
     if ( !m_xUnoControl.is() || !isAliveMode( m_xUnoControl ) )
@@ -316,13 +295,11 @@ void SAL_CALL AccessibleControlShape::grabFocus(void)  throw (RuntimeException)
     }
 }
 
-//-----------------------------------------------------------------------------
 OUString SAL_CALL AccessibleControlShape::getImplementationName(void) throw (RuntimeException)
 {
     return OUString( "com.sun.star.comp.accessibility.AccessibleControlShape" );
 }
 
-//-----------------------------------------------------------------------------
 OUString AccessibleControlShape::CreateAccessibleBaseName(void) throw (RuntimeException)
 {
     OUString sName;
@@ -346,7 +323,6 @@ OUString AccessibleControlShape::CreateAccessibleBaseName(void) throw (RuntimeEx
 
 
 
-//--------------------------------------------------------------------
 OUString
     AccessibleControlShape::CreateAccessibleDescription (void)
     throw (RuntimeException)
@@ -383,11 +359,9 @@ OUString
     return aDG();
 }
 
-//--------------------------------------------------------------------
 IMPLEMENT_FORWARD_REFCOUNT( AccessibleControlShape, AccessibleShape )
 IMPLEMENT_GET_IMPLEMENTATION_ID( AccessibleControlShape )
 
-//--------------------------------------------------------------------
 void SAL_CALL AccessibleControlShape::propertyChange( const PropertyChangeEvent& _rEvent ) throw (RuntimeException)
 {
     ::osl::MutexGuard aGuard( maMutex );
@@ -414,7 +388,6 @@ void SAL_CALL AccessibleControlShape::propertyChange( const PropertyChangeEvent&
 #endif
 }
 
-//--------------------------------------------------------------------
 Any SAL_CALL AccessibleControlShape::queryInterface( const Type& _rType ) throw (RuntimeException)
 {
     Any aReturn = AccessibleShape::queryInterface( _rType );
@@ -427,7 +400,6 @@ Any SAL_CALL AccessibleControlShape::queryInterface( const Type& _rType ) throw
     return aReturn;
 }
 
-//--------------------------------------------------------------------
 Sequence< Type > SAL_CALL AccessibleControlShape::getTypes() throw (RuntimeException)
 {
     Sequence< Type > aShapeTypes = AccessibleShape::getTypes();
@@ -457,7 +429,6 @@ Sequence< Type > SAL_CALL AccessibleControlShape::getTypes() throw (RuntimeExcep
     return aAllTypes;
 }
 
-//--------------------------------------------------------------------
 void SAL_CALL AccessibleControlShape::notifyEvent( const AccessibleEventObject& _rEvent ) throw (RuntimeException)
 {
     if ( AccessibleEventId::STATE_CHANGED == _rEvent.EventId )
@@ -493,7 +464,6 @@ void SAL_CALL AccessibleControlShape::notifyEvent( const AccessibleEventObject&
     }
 }
 
-//--------------------------------------------------------------------
 void SAL_CALL AccessibleControlShape::modeChanged( const ModeChangeEvent& _rSource ) throw (RuntimeException)
 {
     // did it come from our inner context (the real one, not it's proxy!)?
@@ -514,13 +484,11 @@ void SAL_CALL AccessibleControlShape::modeChanged( const ModeChangeEvent& _rSour
 #endif
 }
 
-//--------------------------------------------------------------------
 void SAL_CALL AccessibleControlShape::disposing (const EventObject& _rSource) throw (RuntimeException)
 {
     AccessibleShape::disposing( _rSource );
 }
 
-//--------------------------------------------------------------------
 bool AccessibleControlShape::ensureListeningState(
         const bool _bCurrentlyListening, const bool _bNeedNewListening,
         const OUString& _rPropertyName )
@@ -551,7 +519,6 @@ bool AccessibleControlShape::ensureListeningState(
     return _bNeedNewListening;
 }
 
-//--------------------------------------------------------------------
 sal_Int32 SAL_CALL AccessibleControlShape::getAccessibleChildCount( ) throw(RuntimeException)
 {
     if ( !m_xUnoControl.is() )
@@ -569,7 +536,6 @@ sal_Int32 SAL_CALL AccessibleControlShape::getAccessibleChildCount( ) throw(Runt
     }
 }
 
-//--------------------------------------------------------------------
 Reference< XAccessible > SAL_CALL AccessibleControlShape::getAccessibleChild( sal_Int32 i ) throw(IndexOutOfBoundsException, RuntimeException)
 {
     Reference< XAccessible > xChild;
@@ -613,14 +579,12 @@ Reference< XAccessible > SAL_CALL AccessibleControlShape::getAccessibleChild( sa
     return xChild;
 }
 
-//--------------------------------------------------------------------
 Reference< XAccessibleRelationSet > SAL_CALL AccessibleControlShape::getAccessibleRelationSet(  ) throw (RuntimeException)
 {
     // TODO
     return AccessibleShape::getAccessibleRelationSet( );
 }
 
-//--------------------------------------------------------------------
 OUString AccessibleControlShape::CreateAccessibleName (void) throw (RuntimeException)
 {
     ensureControlModelAccess();
@@ -640,7 +604,6 @@ OUString AccessibleControlShape::CreateAccessibleName (void) throw (RuntimeExcep
     return sName;
 }
 
-//--------------------------------------------------------------------
 void SAL_CALL AccessibleControlShape::disposing (void)
 {
     // ensure we're not listening
@@ -693,7 +656,6 @@ void SAL_CALL AccessibleControlShape::disposing (void)
     AccessibleShape::disposing();
 }
 
-//--------------------------------------------------------------------
 sal_Bool AccessibleControlShape::ensureControlModelAccess() SAL_THROW(())
 {
     if ( m_xControlModel.is() )
@@ -717,7 +679,6 @@ sal_Bool AccessibleControlShape::ensureControlModelAccess() SAL_THROW(())
     return m_xControlModel.is();
 }
 
-//--------------------------------------------------------------------
 void AccessibleControlShape::startStateMultiplexing()
 {
     OSL_PRECOND( !m_bMultiplexingStates, "AccessibleControlShape::startStateMultiplexing: already multiplexing!" );
@@ -738,7 +699,6 @@ void AccessibleControlShape::startStateMultiplexing()
     }
 }
 
-//--------------------------------------------------------------------
 void AccessibleControlShape::stopStateMultiplexing()
 {
     OSL_PRECOND( m_bMultiplexingStates, "AccessibleControlShape::stopStateMultiplexing: not multiplexing!" );
@@ -754,7 +714,6 @@ void AccessibleControlShape::stopStateMultiplexing()
     }
 }
 
-//--------------------------------------------------------------------
 OUString AccessibleControlShape::getControlModelStringProperty( const OUString& _rPropertyName ) const SAL_THROW(())
 {
     OUString sReturn;
@@ -775,7 +734,6 @@ OUString AccessibleControlShape::getControlModelStringProperty( const OUString&
     return sReturn;
 }
 
-//--------------------------------------------------------------------
 void AccessibleControlShape::adjustAccessibleRole( )
 {
     // if we're in design mode, we are a simple SHAPE, in alive mode, we use the role of our inner context
@@ -790,7 +748,7 @@ void AccessibleControlShape::adjustAccessibleRole( )
 }
 
 #ifdef DBG_UTIL
-//--------------------------------------------------------------------
+
 sal_Bool AccessibleControlShape::SetState( sal_Int16 _nState )
 {
     OSL_ENSURE( !isAliveMode( m_xUnoControl ) || !isComposedState( _nState ),
@@ -799,7 +757,6 @@ sal_Bool AccessibleControlShape::SetState( sal_Int16 _nState )
 }
 #endif // DBG_UTIL
 
-//--------------------------------------------------------------------
 void AccessibleControlShape::initializeComposedState()
 {
     if ( !isAliveMode( m_xUnoControl ) )


More information about the Libreoffice-commits mailing list