New loplugin:dynexcspec: Add @throws documentation, chart2

Change-Id: I9b558b713186c64054914ebfc4c122ff769b71a1

15 files changed, 57 insertions, 0 deletions

diff --git a/chart2/inc/ChartModel.hxx b/chart2/inc/ChartModel.hxx
index d45b67fad208..4cbdbfd5defa 100644
--- a/chart2/inc/ChartModel.hxx
+++ b/chart2/inc/ChartModel.hxx
@@ -179,16 +179,20 @@ private:
     bool
         impl_isControllerConnected( const css::uno::Reference< com::sun::star::frame::XController >& xController );
 
+    /// @throws css::uno::RuntimeException
     css::uno::Reference< css::frame::XController >
         impl_getCurrentController()
                             throw( css::uno::RuntimeException);
 
+    /// @throws css::uno::RuntimeException
     void SAL_CALL
         impl_notifyModifiedListeners()
                             throw( css::uno::RuntimeException);
+    /// @throws css::uno::RuntimeException
     void SAL_CALL
         impl_notifyCloseListeners()
                             throw( css::uno::RuntimeException);
+    /// @throws css::uno::RuntimeException
     void SAL_CALL
         impl_notifyStorageChangeListeners()
                             throw(css::uno::RuntimeException);
diff --git a/chart2/source/controller/chartapiwrapper/TitleWrapper.hxx b/chart2/source/controller/chartapiwrapper/TitleWrapper.hxx
index 9eb67a824604..4017385040b9 100644
--- a/chart2/source/controller/chartapiwrapper/TitleWrapper.hxx
+++ b/chart2/source/controller/chartapiwrapper/TitleWrapper.hxx
@@ -91,6 +91,7 @@ protected:
 
     // character properties have to be handled differently (via the XFormattedString elements)
     void getFastCharacterPropertyValue( sal_Int32 nHandle, css::uno::Any& rValue );
+    /// @throws css::uno::Exception
     void setFastCharacterPropertyValue( sal_Int32 nHandle, const css::uno::Any& rValue )
         throw (css::uno::Exception);
 
diff --git a/chart2/source/controller/chartapiwrapper/WrappedScaleProperty.hxx b/chart2/source/controller/chartapiwrapper/WrappedScaleProperty.hxx
index 4722e516f8d1..164731da68ca 100644
--- a/chart2/source/controller/chartapiwrapper/WrappedScaleProperty.hxx
+++ b/chart2/source/controller/chartapiwrapper/WrappedScaleProperty.hxx
@@ -66,8 +66,16 @@ public:
                         throw (css::beans::UnknownPropertyException, css::lang::WrappedTargetException, css::uno::RuntimeException) override;
 
 protected: //methods
+    /// @throws css::beans::UnknownPropertyException
+    /// @throws css::beans::PropertyVetoException
+    /// @throws css::lang::IllegalArgumentException
+    /// @throws css::lang::WrappedTargetException
+    /// @throws css::uno::RuntimeException
     void setPropertyValue( tScaleProperty eScaleProperty, const css::uno::Any& rOuterValue, const css::uno::Reference< css::beans::XPropertySet >& xInnerPropertySet ) const
                         throw (css::beans::UnknownPropertyException, css::beans::PropertyVetoException, css::lang::IllegalArgumentException, css::lang::WrappedTargetException, css::uno::RuntimeException);
+    /// @throws css::beans::UnknownPropertyException
+    /// @throws css::lang::WrappedTargetException
+    /// @throws css::uno::RuntimeException
     css::uno::Any getPropertyValue( tScaleProperty eScaleProperty, const css::uno::Reference< css::beans::XPropertySet >& xInnerPropertySet ) const
                         throw (css::beans::UnknownPropertyException, css::lang::WrappedTargetException, css::uno::RuntimeException);
 
diff --git a/chart2/source/controller/dialogs/ChartTypeDialogController.hxx b/chart2/source/controller/dialogs/ChartTypeDialogController.hxx
index ec36f07ddff0..39647f97a580 100644
--- a/chart2/source/controller/dialogs/ChartTypeDialogController.hxx
+++ b/chart2/source/controller/dialogs/ChartTypeDialogController.hxx
@@ -111,6 +111,7 @@ public:
     virtual void    fillExtraControls( const ChartTypeParameter& rParameter
                                      , const css::uno::Reference< css::chart2::XChartDocument >& xChartModel
                                      , const css::uno::Reference< css::beans::XPropertySet >& xTemplateProps ) const;
+    /// @throws css::uno::RuntimeException
     virtual void    setTemplateProperties( const css::uno::Reference< css::beans::XPropertySet >& xTemplateProps ) const throw (css::uno::RuntimeException);
 
     bool                isSubType( const OUString& rServiceName );
diff --git a/chart2/source/controller/inc/AccessibleBase.hxx b/chart2/source/controller/inc/AccessibleBase.hxx
index dd95d8a0dc60..d822c39111bc 100644
--- a/chart2/source/controller/inc/AccessibleBase.hxx
+++ b/chart2/source/controller/inc/AccessibleBase.hxx
@@ -113,6 +113,7 @@ protected:
                object is already disposed
         @return true, if the component is already disposed and bThrowException is false,
                 false otherwise
+        @throws css::lang::DisposedException
      */
     bool             CheckDisposeState( bool bThrowException = true ) const throw (css::lang::DisposedException);
 
@@ -132,11 +133,15 @@ protected:
     bool     NotifyEvent( EventType eType, const AccessibleUniqueId & rId );
 
     /** Adds a state to the set.
+
+        @throws css::uno::RuntimeException
     */
     void             AddState( sal_Int16 aState ) throw (css::uno::RuntimeException);
 
     /** Removes a state from the set if the set contains the state, otherwise
         nothing is done.
+
+        @throws css::uno::RuntimeException
     */
     void             RemoveState( sal_Int16 aState ) throw (css::uno::RuntimeException);
 
@@ -200,6 +205,9 @@ protected:
 
     /** Is called from getAccessibleChild(). Before this method is called, an
         update of children is done if necessary.
+
+        @throws css::lang::IndexOutOfBoundsException
+        @throws css::uno::RuntimeException
      */
     virtual css::uno::Reference< css::accessibility::XAccessible >
         ImplGetAccessibleChildById( sal_Int32 i ) const
@@ -208,6 +216,8 @@ protected:
 
     /** Is called from getAccessibleChildCount(). Before this method is called,
         an update of children is done if necessary.
+
+        @throws css::uno::RuntimeException
      */
     virtual sal_Int32 ImplGetAccessibleChildCount() const
         throw (css::uno::RuntimeException);
diff --git a/chart2/source/controller/inc/ChartDocumentWrapper.hxx b/chart2/source/controller/inc/ChartDocumentWrapper.hxx
index c0112a85f1ba..3cf3aa7a8f99 100644
--- a/chart2/source/controller/inc/ChartDocumentWrapper.hxx
+++ b/chart2/source/controller/inc/ChartDocumentWrapper.hxx
@@ -80,6 +80,7 @@ public:
 
     css::uno::Reference< css::drawing::XShapes > getAdditionalShapes() const;
 
+    /// @throws css::uno::RuntimeException
     css::uno::Reference< css::drawing::XDrawPage > impl_getDrawPage() const
         throw (css::uno::RuntimeException);
 
diff --git a/chart2/source/controller/inc/ItemConverter.hxx b/chart2/source/controller/inc/ItemConverter.hxx
index df54e141a37c..5763318669eb 100644
--- a/chart2/source/controller/inc/ItemConverter.hxx
+++ b/chart2/source/controller/inc/ItemConverter.hxx
@@ -145,6 +145,8 @@ protected:
         false.
 
         The default implementation does nothing except showing an assertion
+
+        @throws css::uno::Exception
      */
     virtual void FillSpecialItem( sal_uInt16 nWhichId, SfxItemSet & rOutItemSet ) const
         throw (css::uno::Exception, std::exception);
@@ -157,6 +159,8 @@ protected:
         The default implementation returns just false and shows an assertion
 
         @return true if the item changed a property, false otherwise.
+
+        @throws css::uno::Exception
      */
     virtual bool ApplySpecialItem( sal_uInt16 nWhichId, const SfxItemSet & rItemSet )
         throw( css::uno::Exception );
diff --git a/chart2/source/inc/LifeTime.hxx b/chart2/source/inc/LifeTime.hxx
index 64f33af83584..ba652671103f 100644
--- a/chart2/source/inc/LifeTime.hxx
+++ b/chart2/source/inc/LifeTime.hxx
@@ -43,6 +43,7 @@ public:
     virtual ~LifeTimeManager();
 
     bool        impl_isDisposed( bool bAssert=true );
+    /// @throws css::uno::RuntimeException
     bool    dispose() throw(css::uno::RuntimeException);
 
 public:
@@ -89,12 +90,15 @@ OOO_DLLPUBLIC_CHARTTOOLS    CloseableLifeTimeManager( css::util::XCloseable* pCl
 OOO_DLLPUBLIC_CHARTTOOLS    virtual ~CloseableLifeTimeManager() override;
 
 OOO_DLLPUBLIC_CHARTTOOLS    bool        impl_isDisposedOrClosed( bool bAssert=true );
+/// @throws css::uno::Exception
 OOO_DLLPUBLIC_CHARTTOOLS    bool    g_close_startTryClose(bool bDeliverOwnership)
                     throw ( css::uno::Exception );
+/// @throws css::util::CloseVetoException
 OOO_DLLPUBLIC_CHARTTOOLS    bool    g_close_isNeedToCancelLongLastingCalls( bool bDeliverOwnership, css::util::CloseVetoException& ex )
                     throw ( css::util::CloseVetoException );
 OOO_DLLPUBLIC_CHARTTOOLS    void        g_close_endTryClose(bool bDeliverOwnership, bool bMyVeto );
 OOO_DLLPUBLIC_CHARTTOOLS    void        g_close_endTryClose_doClose();
+/// @throws css::uno::RuntimeException
 OOO_DLLPUBLIC_CHARTTOOLS    void    g_addCloseListener( const css::uno::Reference< css::util::XCloseListener > & xListener )
                     throw(css::uno::RuntimeException);
 
diff --git a/chart2/source/inc/OPropertySet.hxx b/chart2/source/inc/OPropertySet.hxx
index a5c7de8eb602..0d0f5fa02157 100644
--- a/chart2/source/inc/OPropertySet.hxx
+++ b/chart2/source/inc/OPropertySet.hxx
@@ -64,6 +64,9 @@ protected:
     /** implement this method to provide default values for all properties
         supporting defaults.  If a property does not have a default value, you
         may throw an UnknownPropertyException.
+
+        @throws css::beans::UnknownPropertyException
+        @throws css::uno::RuntimeException
      */
     virtual css::uno::Any GetDefaultValue( sal_Int32 nHandle ) const
         throw (css::beans::UnknownPropertyException,
diff --git a/chart2/source/inc/WrappedProperty.hxx b/chart2/source/inc/WrappedProperty.hxx
index 062d79566e34..c22a0cb828ef 100644
--- a/chart2/source/inc/WrappedProperty.hxx
+++ b/chart2/source/inc/WrappedProperty.hxx
@@ -40,18 +40,33 @@ public:
     const OUString& getOuterName() const { return m_aOuterName;}
     virtual OUString getInnerName() const;
 
+    /// @throws css::beans::UnknownPropertyException
+    /// @throws css::beans::PropertyVetoException
+    /// @throws css::lang::IllegalArgumentException
+    /// @throws css::lang::WrappedTargetException
+    /// @throws css::uno::RuntimeException
     virtual void setPropertyValue( const css::uno::Any& rOuterValue, const css::uno::Reference< css::beans::XPropertySet >& xInnerPropertySet ) const
                         throw (css::beans::UnknownPropertyException, css::beans::PropertyVetoException, css::lang::IllegalArgumentException, css::lang::WrappedTargetException, css::uno::RuntimeException);
 
+    /// @throws css::beans::UnknownPropertyException
+    /// @throws css::lang::WrappedTargetException
+    /// @throws css::uno::RuntimeException
     virtual css::uno::Any getPropertyValue( const css::uno::Reference< css::beans::XPropertySet >& xInnerPropertySet ) const
                         throw (css::beans::UnknownPropertyException, css::lang::WrappedTargetException, css::uno::RuntimeException);
 
+    /// @throws css::beans::UnknownPropertyException
+    /// @throws css::uno::RuntimeException
     virtual void setPropertyToDefault( const css::uno::Reference< css::beans::XPropertyState >& xInnerPropertyState ) const
                         throw (css::beans::UnknownPropertyException, css::uno::RuntimeException);
 
+    /// @throws css::beans::UnknownPropertyException
+    /// @throws css::lang::WrappedTargetException
+    /// @throws css::uno::RuntimeException
     virtual css::uno::Any getPropertyDefault( const css::uno::Reference< css::beans::XPropertyState >& xInnerPropertyState ) const
                         throw (css::beans::UnknownPropertyException, css::lang::WrappedTargetException, css::uno::RuntimeException);
 
+    /// @throws css::beans::UnknownPropertyException
+    /// @throws css::uno::RuntimeException
     virtual css::beans::PropertyState getPropertyState( const css::uno::Reference< css::beans::XPropertyState >& xInnerPropertyState ) const
                         throw (css::beans::UnknownPropertyException, css::uno::RuntimeException);
 
diff --git a/chart2/source/model/template/ChartTypeTemplate.hxx b/chart2/source/model/template/ChartTypeTemplate.hxx
index 3ae8a9a9a91c..6bc9fae1efc6 100644
--- a/chart2/source/model/template/ChartTypeTemplate.hxx
+++ b/chart2/source/model/template/ChartTypeTemplate.hxx
@@ -114,6 +114,7 @@ protected:
         const css::uno::Reference< css::chart2::XDiagram >& xDiagram )
         throw (css::uno::RuntimeException, std::exception) override;
 
+    /// @throws css::uno::RuntimeException
     void SAL_CALL applyStyles(
         const css::uno::Reference< css::chart2::XDiagram >& xDiagram )
         throw (css::uno::RuntimeException);
diff --git a/chart2/source/view/axes/VAxisOrGridBase.hxx b/chart2/source/view/axes/VAxisOrGridBase.hxx
index c8884d8acc5a..be81876a06de 100644
--- a/chart2/source/view/axes/VAxisOrGridBase.hxx
+++ b/chart2/source/view/axes/VAxisOrGridBase.hxx
@@ -40,6 +40,7 @@ public:
     virtual ~VAxisOrGridBase() override;
 
     virtual void setTransformationSceneToScreen( const css::drawing::HomogenMatrix& rMatrix ) override;
+    /// @throws css::uno::RuntimeException
     virtual void setExplicitScaleAndIncrement(
             const ExplicitScaleData& rScale
           , const ExplicitIncrementData& rIncrement )
diff --git a/chart2/source/view/inc/PlotterBase.hxx b/chart2/source/view/inc/PlotterBase.hxx
index de4c9790531c..1d12d8633e56 100644
--- a/chart2/source/view/inc/PlotterBase.hxx
+++ b/chart2/source/view/inc/PlotterBase.hxx
@@ -48,6 +48,7 @@ public:
     PlotterBase( sal_Int32 nDimension );
     virtual ~PlotterBase();
 
+    /// @throws css::uno::RuntimeException
     virtual void initPlotter(
           const css::uno::Reference< css::drawing::XShapes >& xLogicTarget
         , const css::uno::Reference< css::drawing::XShapes >& xFinalTarget
diff --git a/chart2/source/view/inc/VCoordinateSystem.hxx b/chart2/source/view/inc/VCoordinateSystem.hxx
index 53828536e999..aac86229d1b9 100644
--- a/chart2/source/view/inc/VCoordinateSystem.hxx
+++ b/chart2/source/view/inc/VCoordinateSystem.hxx
@@ -50,6 +50,7 @@ public:
     static VCoordinateSystem* createCoordinateSystem( const css::uno::Reference<
                                 css::chart2::XCoordinateSystem >& xCooSysModel );
 
+    /// @throws css::uno::RuntimeException
     void initPlottingTargets(
                   const css::uno::Reference< css::drawing::XShapes >& xLogicTarget
                 , const css::uno::Reference< css::drawing::XShapes >& xFinalTarget
diff --git a/chart2/source/view/inc/VSeriesPlotter.hxx b/chart2/source/view/inc/VSeriesPlotter.hxx
index cd8cf9ffd6d3..470de99d98cb 100644
--- a/chart2/source/view/inc/VSeriesPlotter.hxx
+++ b/chart2/source/view/inc/VSeriesPlotter.hxx
@@ -152,6 +152,8 @@ public:
     The parameter nAxisIndex matches this DataSeries property 'AttachedAxisIndex'.
     nAxisIndex must be greater than 0. nAxisIndex==1 referres to the first secondary axis.
     )
+
+    @throws css::uno::RuntimeException
     */
 
     void addSecondaryValueScale( const ExplicitScaleData& rScale, sal_Int32 nAxisIndex )