diff options
author | Noel Grandin <noel@peralex.com> | 2016-04-20 10:34:01 +0200 |
---|---|---|
committer | Noel Grandin <noel@peralex.com> | 2016-04-21 08:32:47 +0200 |
commit | 5abc669599001bf888b97c4d3c2715e1fb7523b9 (patch) | |
tree | 2407c6fc040a795e6ffc69de02ba940285c04c7f /vbahelper | |
parent | 5bb308a9ad16f6002486a60e4a753693818580b6 (diff) |
new plugin stylepolice
check for local variables which follow our member field naming
convention, which is highly confusing
Change-Id: Idacedf7145d09843e96a584237b385f7662eea10
Diffstat (limited to 'vbahelper')
-rw-r--r-- | vbahelper/source/vbahelper/vbadocumentsbase.cxx | 6 | ||||
-rw-r--r-- | vbahelper/source/vbahelper/vbashapes.cxx | 14 |
2 files changed, 10 insertions, 10 deletions
diff --git a/vbahelper/source/vbahelper/vbadocumentsbase.cxx b/vbahelper/source/vbahelper/vbadocumentsbase.cxx index ca8aa5e41b12..230a7f022b1d 100644 --- a/vbahelper/source/vbahelper/vbadocumentsbase.cxx +++ b/vbahelper/source/vbahelper/vbadocumentsbase.cxx @@ -80,10 +80,10 @@ public: explicit DocumentsEnumImpl( const uno::Reference< uno::XComponentContext >& xContext ) throw ( uno::RuntimeException ) : m_xContext( xContext ) { uno::Reference< frame::XDesktop2 > xDesktop = frame::Desktop::create( m_xContext ); - uno::Reference< container::XEnumeration > mxComponents = xDesktop->getComponents()->createEnumeration(); - while( mxComponents->hasMoreElements() ) + uno::Reference< container::XEnumeration > xComponents = xDesktop->getComponents()->createEnumeration(); + while( xComponents->hasMoreElements() ) { - uno::Reference< frame::XModel > xNext( mxComponents->nextElement(), uno::UNO_QUERY ); + uno::Reference< frame::XModel > xNext( xComponents->nextElement(), uno::UNO_QUERY ); if ( xNext.is() ) m_documents.push_back( xNext ); } diff --git a/vbahelper/source/vbahelper/vbashapes.cxx b/vbahelper/source/vbahelper/vbashapes.cxx index 0126bfebabd4..fc66a1747c49 100644 --- a/vbahelper/source/vbahelper/vbashapes.cxx +++ b/vbahelper/source/vbahelper/vbashapes.cxx @@ -68,12 +68,12 @@ void ScVbaShapes::initBaseCollection() if ( m_xNameAccess.is() ) // already has NameAccess return; // no NameAccess then use ShapeCollectionHelper - XNamedObjectCollectionHelper< drawing::XShape >::XNamedVec mShapes; + XNamedObjectCollectionHelper< drawing::XShape >::XNamedVec aShapes; sal_Int32 nLen = m_xIndexAccess->getCount(); - mShapes.reserve( nLen ); + aShapes.reserve( nLen ); for ( sal_Int32 index=0; index<nLen; ++index ) - mShapes.push_back( uno::Reference< drawing::XShape >( m_xIndexAccess->getByIndex( index ) , uno::UNO_QUERY ) ); - uno::Reference< container::XIndexAccess > xShapes( new XNamedObjectCollectionHelper< drawing::XShape >( mShapes ) ); + aShapes.push_back( uno::Reference< drawing::XShape >( m_xIndexAccess->getByIndex( index ) , uno::UNO_QUERY ) ); + uno::Reference< container::XIndexAccess > xShapes( new XNamedObjectCollectionHelper< drawing::XShape >( aShapes ) ); m_xIndexAccess.set( xShapes, uno::UNO_QUERY ); m_xNameAccess.set( xShapes, uno::UNO_QUERY ); } @@ -138,7 +138,7 @@ ScVbaShapes::getShapesByArrayIndices( const uno::Any& Index ) throw (uno::Runti uno::Sequence< uno::Any > sIndices; aConverted >>= sIndices; - XNamedObjectCollectionHelper< drawing::XShape >::XNamedVec mShapes; + XNamedObjectCollectionHelper< drawing::XShape >::XNamedVec aShapes; sal_Int32 nElems = sIndices.getLength(); for( sal_Int32 index = 0; index < nElems; ++index ) { @@ -159,9 +159,9 @@ ScVbaShapes::getShapesByArrayIndices( const uno::Any& Index ) throw (uno::Runti } // populate map with drawing::XShapes if ( xShape.is() ) - mShapes.push_back( xShape ); + aShapes.push_back( xShape ); } - uno::Reference< container::XIndexAccess > xIndexAccess( new XNamedObjectCollectionHelper< drawing::XShape >( mShapes ) ); + uno::Reference< container::XIndexAccess > xIndexAccess( new XNamedObjectCollectionHelper< drawing::XShape >( aShapes ) ); return xIndexAccess; } |