[Libreoffice-commits] .: 2 commits - chart2/workbench sc/source

Caolán McNamara caolan at kemper.freedesktop.org
Wed Apr 13 12:38:41 PDT 2011


 chart2/workbench/addin/sampleaddin.cxx |    4 ++--
 sc/source/ui/vba/vbachart.cxx          |   30 +++++++++++++++---------------
 2 files changed, 17 insertions(+), 17 deletions(-)

New commits:
commit 4a8f0b019090fb630c331212c6ae4d5efed31f83
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Wed Apr 13 11:26:26 2011 +0100

    catch by const reference

diff --git a/sc/source/ui/vba/vbachart.cxx b/sc/source/ui/vba/vbachart.cxx
index fc91f1d..dbbae06 100644
--- a/sc/source/ui/vba/vbachart.cxx
+++ b/sc/source/ui/vba/vbachart.cxx
@@ -96,7 +96,7 @@ ScVbaChart::getName() throw (css::uno::RuntimeException)
     {
         xProps->getPropertyValue( CHART_NAME ) >>= sName;
     }
-    catch( uno::Exception & ) // swallow exceptions
+    catch( const uno::Exception & ) // swallow exceptions
     {
     }
     return sName;
@@ -220,7 +220,7 @@ ScVbaChart::getChartType() throw ( uno::RuntimeException, script::BasicErrorExce
             nChartType = getMarkerType(xlRadarMarkers, xlRadar);
         }
     }
-    catch (uno::Exception& )
+    catch ( const uno::Exception& )
     {
         throw script::BasicErrorException( rtl::OUString(), uno::Reference< uno::XInterface >(), SbERR_METHOD_FAILED, rtl::OUString() );
     }
@@ -547,7 +547,7 @@ try
                 break;
         }
     }
-    catch ( uno::Exception& )
+    catch ( const uno::Exception& )
     {
         throw script::BasicErrorException( rtl::OUString(), uno::Reference< uno::XInterface >(), SbERR_METHOD_FAILED, rtl::OUString() );
     }
@@ -630,7 +630,7 @@ ScVbaChart::setSourceData( const css::uno::Reference< ::ooo::vba::excel::XRange
                 setPlotBy( xlRows );
         }
     }
-    catch (uno::Exception& )
+    catch (const uno::Exception&)
     {
         throw script::BasicErrorException( rtl::OUString(), uno::Reference< uno::XInterface >(), SbERR_METHOD_FAILED, rtl::OUString() );
     }
@@ -673,7 +673,7 @@ ScVbaChart::setPlotBy( ::sal_Int32 _nPlotBy ) throw (css::script::BasicErrorExce
                 throw script::BasicErrorException( rtl::OUString(), uno::Reference< uno::XInterface >(), SbERR_METHOD_FAILED, rtl::OUString() );
         }
     }
-    catch (uno::Exception& )
+    catch (const uno::Exception&)
     {
         throw script::BasicErrorException( rtl::OUString(), uno::Reference< uno::XInterface >(), SbERR_METHOD_FAILED, rtl::OUString() );
     }	
@@ -695,7 +695,7 @@ ScVbaChart::getPlotBy(  ) throw (script::BasicErrorException, uno::RuntimeExcept
             return xlRows;
         }
     }
-    catch (uno::Exception& )
+    catch (const uno::Exception&)
     {
         throw script::BasicErrorException( rtl::OUString(), uno::Reference< uno::XInterface >(), SbERR_METHOD_FAILED, rtl::OUString() );
     }	
@@ -711,7 +711,7 @@ ScVbaChart::setDiagram( const rtl::OUString& _sDiagramType ) throw( script::Basi
         mxChartDocument->setDiagram( xDiagram );
         mxDiagramPropertySet.set( xDiagram, uno::UNO_QUERY_THROW );
     }
-    catch ( uno::Exception& )
+    catch ( const uno::Exception& )
     {
         throw script::BasicErrorException( rtl::OUString(), uno::Reference< uno::XInterface >(), SbERR_METHOD_FAILED, rtl::OUString() );
     }
@@ -827,7 +827,7 @@ ScVbaChart::getHasTitle(  ) throw (script::BasicErrorException, uno::RuntimeExce
     {
         mxChartPropertySet->getPropertyValue(HASMAINTITLE) >>= bHasTitle;
     }
-    catch (uno::Exception& )
+    catch (const uno::Exception&)
     {
         throw script::BasicErrorException( rtl::OUString(), uno::Reference< uno::XInterface >(), SbERR_METHOD_FAILED, rtl::OUString() );
     }
@@ -841,7 +841,7 @@ ScVbaChart::setHasTitle( ::sal_Bool bTitle ) throw (script::BasicErrorException,
     {
         mxChartPropertySet->setPropertyValue(HASMAINTITLE, uno::makeAny( bTitle ));
     }
-    catch (uno::Exception& )
+    catch (const uno::Exception&)
     {
         throw script::BasicErrorException( rtl::OUString(), uno::Reference< uno::XInterface >(), SbERR_METHOD_FAILED, rtl::OUString() );
     }
@@ -856,7 +856,7 @@ ScVbaChart::getHasLegend(  ) throw (script::BasicErrorException, uno::RuntimeExc
     {
         mxChartPropertySet->getPropertyValue(HASLEGEND) >>= bHasLegend;
     }
-    catch (uno::Exception& )
+    catch (const uno::Exception&)
     {
         throw script::BasicErrorException( rtl::OUString(), uno::Reference< uno::XInterface >(), SbERR_METHOD_FAILED, rtl::OUString() );
     }
@@ -870,7 +870,7 @@ ScVbaChart::setHasLegend( ::sal_Bool bLegend ) throw (script::BasicErrorExceptio
     {
         mxChartPropertySet->setPropertyValue(HASLEGEND, uno::makeAny(bLegend));
     }
-    catch (uno::Exception& )
+    catch (const uno::Exception&)
     {
         throw script::BasicErrorException( rtl::OUString(), uno::Reference< uno::XInterface >(), SbERR_METHOD_FAILED, rtl::OUString() );
     }
@@ -961,7 +961,7 @@ ScVbaChart::getSolidType(sal_Int32 _nDeep, sal_Int32 _nVertiStacked, sal_Int32 _
             }
         }
     }
-    catch (uno::Exception& )
+    catch (const uno::Exception&)
     {
         throw script::BasicErrorException( rtl::OUString(), uno::Reference< uno::XInterface >(), SbERR_METHOD_FAILED, rtl::OUString() );
     }
@@ -984,7 +984,7 @@ ScVbaChart::getStockUpDownValue(sal_Int32 _nUpDown, sal_Int32 _nNotUpDown) throw
             return _nNotUpDown;
         }
     }
-    catch (uno::Exception& )
+    catch (const uno::Exception&)
     {
         rtl::OUString aTemp;    // temporary needed for g++ 3.3.5
         script::BasicErrorException( aTemp, uno::Reference< uno::XInterface >(), SbERR_METHOD_FAILED, rtl::OUString() );	
@@ -1002,7 +1002,7 @@ ScVbaChart::hasMarkers() throw ( script::BasicErrorException )
         mxDiagramPropertySet->getPropertyValue(SYMBOLTYPE) >>= nSymbol;
         bHasMarkers = nSymbol != chart::ChartSymbolType::NONE;
     }
-    catch ( uno::Exception& )
+    catch (const uno::Exception&)
     {
         rtl::OUString aTemp;    // temporary needed for g++ 3.3.5
         script::BasicErrorException( aTemp, uno::Reference< uno::XInterface >(), SbERR_METHOD_FAILED, rtl::OUString() );	
@@ -1050,7 +1050,7 @@ ScVbaChart::isSeriesIndexValid(sal_Int32 _seriesindex) throw( script::BasicError
             }
         }
     } 
-    catch (uno::Exception& ) 
+    catch (const uno::Exception&)
     {
         throw script::BasicErrorException( rtl::OUString(), uno::Reference< uno::XInterface >(), SbERR_METHOD_FAILED, rtl::OUString() );
     }
commit 65544bd46cf7e11d9d6c3c635c1faee6d3a480bd
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Wed Apr 13 11:22:34 2011 +0100

    catch by const reference

diff --git a/chart2/workbench/addin/sampleaddin.cxx b/chart2/workbench/addin/sampleaddin.cxx
index c15ef3a..49ad6da 100644
--- a/chart2/workbench/addin/sampleaddin.cxx
+++ b/chart2/workbench/addin/sampleaddin.cxx
@@ -71,7 +71,7 @@ sal_Bool SAL_CALL component_writeInfo(
 
             return sal_True;
         }
-        catch( registry::InvalidRegistryException& )
+        catch( const registry::InvalidRegistryException& )
         {
             OSL_FAIL( "### InvalidRegistryException!" );
         }
@@ -163,7 +163,7 @@ sal_Bool SampleAddIn::getLogicalPosition( uno::Reference< drawing::XShape >& xAx
                     bRet = sal_True;
                 }
             }
-            catch( beans::UnknownPropertyException )
+            catch( const beans::UnknownPropertyException& )
             {
                 // the shape xAxis was no chart axis
             }


More information about the Libreoffice-commits mailing list