New loplugin:dynexcspec: Add @throws documentation, svx

Change-Id: I0e827554a18622e19aec5f81c40da52664783f69

24 files changed, 64 insertions, 4 deletions

diff --git a/svx/inc/AccessibleTableShape.hxx b/svx/inc/AccessibleTableShape.hxx
index 2adcc0618332..e0e0ac465135 100644
--- a/svx/inc/AccessibleTableShape.hxx
+++ b/svx/inc/AccessibleTableShape.hxx
@@ -125,6 +125,7 @@ public:
     using AccessibleShape::disposing;
     friend class AccessibleTableHeaderShape;
 
+    /// @throws css::lang::IndexOutOfBoundsException
     void getColumnAndRow( sal_Int32 nChildIndex, sal_Int32& rnColumn, sal_Int32& rnRow ) throw (css::lang::IndexOutOfBoundsException );
     // overwrite the SetState & ResetState to do special operation for table cell's internal text
     virtual bool SetState (sal_Int16 aState) override;
@@ -140,6 +141,7 @@ protected:
 
     sdr::table::SvxTableController* getTableController();
 
+    /// @throws css::lang::IndexOutOfBoundsException
     void checkCellPosition( sal_Int32 nCol, sal_Int32 nRow ) throw ( css::lang::IndexOutOfBoundsException );
 
 private:
diff --git a/svx/source/accessibility/ChildrenManagerImpl.hxx b/svx/source/accessibility/ChildrenManagerImpl.hxx
index d8c0776bb195..27d73b74034c 100644
--- a/svx/source/accessibility/ChildrenManagerImpl.hxx
+++ b/svx/source/accessibility/ChildrenManagerImpl.hxx
@@ -118,6 +118,8 @@ public:
     */
     long GetChildCount() const throw ();
 
+    /// @throws css::uno::RuntimeException
+    /// @throws css::lang::IndexOutOfBoundsException
     css::uno::Reference<css::drawing::XShape> GetChildShape(long nIndex)
         throw (css::uno::RuntimeException,
                css::lang::IndexOutOfBoundsException);
@@ -149,6 +151,7 @@ public:
             Returns a reference to the requested accessible child.  This
             reference is empty if it has not been possible to create the
             accessible object of the corresponding shape.
+        @throws css::uno::RuntimeException
     */
     css::uno::Reference<css::accessibility::XAccessible>
         GetChild (ChildDescriptor& aChildDescriptor,sal_Int32 _nIndex)
@@ -165,6 +168,7 @@ public:
             Returns a reference to the requested accessible child.  The
             reference is empty if there is no shape descriptor that
             associates the shape with an accessible object.
+        @throws css::uno::RuntimeException
     */
     css::uno::Reference<css::accessibility::XAccessible>
         GetChild (const css::uno::Reference<css::drawing::XShape>& xShape)
diff --git a/svx/source/form/filtnav.cxx b/svx/source/form/filtnav.cxx
index a8d8aa2b4e10..3987b458315e 100644
--- a/svx/source/form/filtnav.cxx
+++ b/svx/source/form/filtnav.cxx
@@ -250,6 +250,7 @@ public:
     virtual void SAL_CALL disjunctiveTermAdded( const FilterEvent& Event ) throw (RuntimeException, std::exception) override;
 
 // helpers
+    /// @throws RuntimeException
     void dispose() throw( RuntimeException );
 
     void AddOrRemoveListener( const Reference< XIndexAccess >& _rxControllers, const bool _bAdd );
diff --git a/svx/source/form/fmundo.cxx b/svx/source/form/fmundo.cxx
index 1a05c6910e3e..6963b0db8cee 100644
--- a/svx/source/form/fmundo.cxx
+++ b/svx/source/form/fmundo.cxx
@@ -78,6 +78,7 @@ using namespace ::dbtools;
 class ScriptEventListenerWrapper : public cppu::WeakImplHelper< XScriptListener >
 {
 public:
+    /// @throws css::uno::RuntimeException
     explicit ScriptEventListenerWrapper( FmFormModel& _rModel) throw ( RuntimeException )
         :m_rModel( _rModel )
         ,m_attemptedListenerCreation( false )
diff --git a/svx/source/inc/AccessibleFrameSelector.hxx b/svx/source/inc/AccessibleFrameSelector.hxx
index 9c2ad91e5205..ddae1082c597 100644
--- a/svx/source/inc/AccessibleFrameSelector.hxx
+++ b/svx/source/inc/AccessibleFrameSelector.hxx
@@ -103,6 +103,7 @@ protected:
     DECL_LINK( WindowEventListener, VclWindowEvent&, void );
 
 private:
+    /// @throws css::uno::RuntimeException
     void                IsValid() throw (css::uno::RuntimeException);
     void                RemoveFrameSelEventListener();
 
diff --git a/svx/source/inc/GraphCtlAccessibleContext.hxx b/svx/source/inc/GraphCtlAccessibleContext.hxx
index c0f3454a930e..1eea3a88a03f 100644
--- a/svx/source/inc/GraphCtlAccessibleContext.hxx
+++ b/svx/source/inc/GraphCtlAccessibleContext.hxx
@@ -158,6 +158,7 @@ public:
     virtual Size LogicToPixel (const Size& rSize) const override;
 
 protected:
+    /// @throws css::lang::IndexOutOfBoundsException
     void checkChildIndexOnSelection( long nIndexOfChild ) throw (css::lang::IndexOutOfBoundsException );
 
 public:
@@ -181,11 +182,15 @@ protected:
      Rectangle GetBoundingBoxOnScreen() throw (css::uno::RuntimeException);
 
     /// Return the object's current bounding box relative to the parent object.
+    ///
+    /// @throws css::uno::RuntimeException
     Rectangle GetBoundingBox() throw (css::uno::RuntimeException);
 
     virtual void SAL_CALL disposing() final override;
 
 private:
+    /// @throws css::uno::RuntimeException
+    /// @throws css::lang::IndexOutOfBoundsException
     SdrObject* getSdrObject( sal_Int32 nIndex )
         throw( css::uno::RuntimeException, css::lang::IndexOutOfBoundsException );
 
diff --git a/svx/source/inc/coreservices.hxx b/svx/source/inc/coreservices.hxx
index 4a83c5126a88..35205a8004d4 100644
--- a/svx/source/inc/coreservices.hxx
+++ b/svx/source/inc/coreservices.hxx
@@ -36,27 +36,39 @@ namespace com { namespace sun { namespace star {
 namespace svx
 {
 OUString SAL_CALL ExtrusionDepthController_getImplementationName();
+/// @throws css::uno::RuntimeException
 css::uno::Reference< css::uno::XInterface > SAL_CALL ExtrusionDepthController_createInstance(const css::uno::Reference< css::lang::XMultiServiceFactory > &)  throw( css::uno::RuntimeException );
+/// @throws css::uno::RuntimeException
 css::uno::Sequence< OUString > SAL_CALL ExtrusionDepthController_getSupportedServiceNames() throw( css::uno::RuntimeException );
 
 OUString SAL_CALL ExtrusionDirectionControl_getImplementationName();
+/// @throws css::uno::RuntimeException
 css::uno::Reference< css::uno::XInterface > SAL_CALL ExtrusionDirectionControl_createInstance(const css::uno::Reference< css::lang::XMultiServiceFactory > &)  throw( css::uno::RuntimeException );
+/// @throws css::uno::RuntimeException
 css::uno::Sequence< OUString > SAL_CALL ExtrusionDirectionControl_getSupportedServiceNames() throw( css::uno::RuntimeException );
 
 OUString SAL_CALL ExtrusionLightingControl_getImplementationName();
+/// @throws css::uno::RuntimeException
 css::uno::Reference< css::uno::XInterface > SAL_CALL ExtrusionLightingControl_createInstance(const css::uno::Reference< css::lang::XMultiServiceFactory > &)  throw( css::uno::RuntimeException );
+/// @throws css::uno::RuntimeException
 css::uno::Sequence< OUString > SAL_CALL ExtrusionLightingControl_getSupportedServiceNames() throw( css::uno::RuntimeException );
 
 OUString SAL_CALL ExtrusionSurfaceControl_getImplementationName();
+/// @throws css::uno::RuntimeException
 css::uno::Reference< css::uno::XInterface > SAL_CALL ExtrusionSurfaceControl_createInstance(const css::uno::Reference< css::lang::XMultiServiceFactory > &)  throw( css::uno::RuntimeException );
+/// @throws css::uno::RuntimeException
 css::uno::Sequence< OUString > SAL_CALL ExtrusionSurfaceControl_getSupportedServiceNames() throw( css::uno::RuntimeException );
 
 OUString SAL_CALL FontworkAlignmentControl_getImplementationName();
+/// @throws css::uno::RuntimeException
 css::uno::Reference< css::uno::XInterface > SAL_CALL FontworkAlignmentControl_createInstance(const css::uno::Reference< css::lang::XMultiServiceFactory > &)  throw( css::uno::RuntimeException );
+/// @throws css::uno::RuntimeException
 css::uno::Sequence< OUString > SAL_CALL FontworkAlignmentControl_getSupportedServiceNames() throw( css::uno::RuntimeException );
 
 OUString SAL_CALL FontworkCharacterSpacingControl_getImplementationName();
+/// @throws css::uno::RuntimeException
 css::uno::Reference< css::uno::XInterface > SAL_CALL FontworkCharacterSpacingControl_createInstance(const css::uno::Reference< css::lang::XMultiServiceFactory > &)  throw( css::uno::RuntimeException );
+/// @throws css::uno::RuntimeException
 css::uno::Sequence< OUString > SAL_CALL FontworkCharacterSpacingControl_getSupportedServiceNames() throw( css::uno::RuntimeException );
 }
 
diff --git a/svx/source/inc/fmservs.hxx b/svx/source/inc/fmservs.hxx
index edc6a016d2bc..60b0dcde5f83 100644
--- a/svx/source/inc/fmservs.hxx
+++ b/svx/source/inc/fmservs.hxx
@@ -92,16 +92,19 @@ namespace svxform
     OAddConditionDialog_GetSupportedServiceNames();
 }
 
+/// @throws css::uno::Exception
 css::uno::Reference<css::uno::XInterface> SAL_CALL
 FmXGridControl_NewInstance_Impl(
     css::uno::Reference<css::lang::XMultiServiceFactory> const &)
     throw (css::uno::Exception);
 
+/// @throws css::uno::Exception
 css::uno::Reference<css::uno::XInterface> SAL_CALL
 FormController_NewInstance_Impl(
     css::uno::Reference<css::lang::XMultiServiceFactory> const &)
     throw (css::uno::Exception);
 
+/// @throws css::uno::Exception
 css::uno::Reference<css::uno::XInterface> SAL_CALL
 LegacyFormController_NewInstance_Impl(
     css::uno::Reference<css::lang::XMultiServiceFactory> const &)
diff --git a/svx/source/inc/formcontroller.hxx b/svx/source/inc/formcontroller.hxx
index 77966e3ea946..9099c0412e47 100644
--- a/svx/source/inc/formcontroller.hxx
+++ b/svx/source/inc/formcontroller.hxx
@@ -476,6 +476,7 @@ namespace svxform
         void addToEventAttacher(const css::uno::Reference< css::awt::XControl>& xControl);
         void removeFromEventAttacher(const css::uno::Reference< css::awt::XControl>& xControl);
         void toggleAutoFields(bool bAutoFields);
+        /// @throws css::uno::RuntimeException
         void unload() throw( css::uno::RuntimeException );
         void removeBoundFieldListener();
 
diff --git a/svx/source/inc/formdispatchinterceptor.hxx b/svx/source/inc/formdispatchinterceptor.hxx
index 73ad565dc550..5708737cf258 100644
--- a/svx/source/inc/formdispatchinterceptor.hxx
+++ b/svx/source/inc/formdispatchinterceptor.hxx
@@ -33,6 +33,7 @@ namespace svxform
     class DispatchInterceptor
     {
     public:
+        /// @throws css::uno::RuntimeException
         virtual css::uno::Reference< css::frame::XDispatch> interceptedQueryDispatch(
             const css::util::URL& aURL, const OUString& aTargetFrameName, sal_Int32 nSearchFlags) throw( css::uno::RuntimeException ) = 0;
 
diff --git a/svx/source/inc/svxpixelctlaccessiblecontext.hxx b/svx/source/inc/svxpixelctlaccessiblecontext.hxx
index 68a379f96a70..7de85a596d7b 100644
--- a/svx/source/inc/svxpixelctlaccessiblecontext.hxx
+++ b/svx/source/inc/svxpixelctlaccessiblecontext.hxx
@@ -130,13 +130,15 @@ public:
     void FireAccessibleEvent (short nEventId, const css::uno::Any& rOld, const css::uno::Any& rNew);
     virtual void SAL_CALL disposing() override;
 
+    /// @throws css::uno::RuntimeException
     Rectangle GetBoundingBoxOnScreen() throw( css::uno::RuntimeException );
 
+    /// @throws css::uno::RuntimeException
     Rectangle const & GetBoundingBox() throw( css::uno::RuntimeException );
 
     /// @returns true if it's disposed or in disposing
     inline bool IsAlive() const;
-    /// throws the exception DisposedException if it's not alive
+    /// @throws css::lang::DisposedException if it's not alive
     void ThrowExceptionIfNotAlive() throw( css::lang::DisposedException );
 
 
diff --git a/svx/source/inc/svxrectctaccessiblecontext.hxx b/svx/source/inc/svxrectctaccessiblecontext.hxx
index 622feca5a7d2..79ce253aec1c 100644
--- a/svx/source/inc/svxrectctaccessiblecontext.hxx
+++ b/svx/source/inc/svxrectctaccessiblecontext.hxx
@@ -101,6 +101,7 @@ public:
     virtual css::awt::Size SAL_CALL
         getSize() throw( css::uno::RuntimeException, std::exception ) override;
 
+    /// @throws css::uno::RuntimeException
     bool SAL_CALL
         isVisible() throw( css::uno::RuntimeException );
 
@@ -200,6 +201,7 @@ public:
 
 protected:
     // internals
+    /// @throws css::lang::IndexOutOfBoundsException
     void checkChildIndex( long nIndexOfChild ) throw( css::lang::IndexOutOfBoundsException );
 
     /** Selects a new child by index.
@@ -227,9 +229,13 @@ public:
 protected:
 
     /// @Return the object's current bounding box relative to the desktop.
+    ///
+    /// @throws css::uno::RuntimeException
     Rectangle GetBoundingBoxOnScreen() throw( css::uno::RuntimeException );
 
     /// @Return the object's current bounding box relative to the parent object.
+    ///
+    /// @throws css::uno::RuntimeException
     Rectangle GetBoundingBox() throw( css::uno::RuntimeException );
 
     virtual void SAL_CALL disposing() override;
@@ -237,7 +243,7 @@ protected:
     /// @returns true if it's disposed or in disposing
     inline bool IsAlive() const;
 
-    /// throws the exception DisposedException if it's not alive
+    /// @throws css::lang::DisposedException if it's not alive
     void ThrowExceptionIfNotAlive() throw( css::lang::DisposedException );
 
 private:
@@ -410,8 +416,10 @@ public:
     void FireFocusEvent();
 
 protected:
+    /// @throws css::uno::RuntimeException
     Rectangle GetBoundingBoxOnScreen() throw( css::uno::RuntimeException );
 
+    /// @throws css::uno::RuntimeException
     Rectangle const & GetBoundingBox() throw( css::uno::RuntimeException );
 
     void CommitChange( const css::accessibility::AccessibleEventObject& rEvent );
@@ -421,7 +429,7 @@ protected:
     /// @returns true if it's disposed or in disposing
     inline bool IsAlive() const;
 
-    /// throws the exception DisposedException if it's not alive
+    /// @throws css::lang::DisposedException if it's not alive
     void ThrowExceptionIfNotAlive() throw( css::lang::DisposedException );
 
     /// Mutex guarding this object.
diff --git a/svx/source/inc/xmlxtexp.hxx b/svx/source/inc/xmlxtexp.hxx
index 1152769ee7a9..71701fa6ede6 100644
--- a/svx/source/inc/xmlxtexp.hxx
+++ b/svx/source/inc/xmlxtexp.hxx
@@ -43,6 +43,7 @@ public:
 
     virtual ~SvxXMLXTableExportComponent() override;
 
+    /// @throws css::uno::RuntimeException
     static bool save( const OUString& rURL,
                       const css::uno::Reference< css::container::XNameContainer >& xTable,
                       const css::uno::Reference< css::embed::XStorage > &xStorage,
diff --git a/svx/source/table/accessibletableshape.cxx b/svx/source/table/accessibletableshape.cxx
index a008b808dc76..58db3b2726cb 100644
--- a/svx/source/table/accessibletableshape.cxx
+++ b/svx/source/table/accessibletableshape.cxx
@@ -69,7 +69,10 @@ public:
     void init( const Reference< XAccessible>& xAccessible, const Reference< XTable >& xTable );
     void dispose();
 
+    /// @throws IndexOutOfBoundsException
+    /// @throws RuntimeException
     Reference< XAccessible > getAccessibleChild(sal_Int32 i) throw (IndexOutOfBoundsException, RuntimeException);
+    /// @throws IndexOutOfBoundsException
     void getColumnAndRow( sal_Int32 nChildIndex, sal_Int32& rnColumn, sal_Int32& rnRow ) throw (IndexOutOfBoundsException );
 
     // XModifyListener
@@ -85,6 +88,8 @@ public:
     sal_Int32 mRowCount, mColCount;
     //get the cached AccessibleCell from XCell
     rtl::Reference< AccessibleCell > getAccessibleCell (const Reference< XCell >& xCell);
+    /// @throws IndexOutOfBoundsException
+    /// @throws RuntimeException
     rtl::Reference< AccessibleCell > getAccessibleCell (sal_Int32 nRow, sal_Int32 nColumn) throw (IndexOutOfBoundsException, RuntimeException);
 };
 
diff --git a/svx/source/table/cell.hxx b/svx/source/table/cell.hxx
index 72f4a15f4eac..572908aa119f 100644
--- a/svx/source/table/cell.hxx
+++ b/svx/source/table/cell.hxx
@@ -198,6 +198,7 @@ protected:
     SVX_DLLPRIVATE static css::uno::Any GetAnyForItem( SfxItemSet& aSet, const SfxItemPropertySimpleEntry* pMap );
 
 private:
+    /// @throws css::uno::RuntimeException
     SVX_DLLPRIVATE Cell( SdrTableObj& rTableObj, OutlinerParaObject* pOutlinerParaObject ) throw(css::uno::RuntimeException);
     SVX_DLLPRIVATE virtual ~Cell() throw() override;
 
diff --git a/svx/source/table/propertyset.hxx b/svx/source/table/propertyset.hxx
index b84125db7206..a88786a32d5e 100644
--- a/svx/source/table/propertyset.hxx
+++ b/svx/source/table/propertyset.hxx
@@ -44,6 +44,7 @@ public:
 
     void addProperties( const PropertyVector& rProps );
 
+    /// @throws css::beans::UnknownPropertyException
     const css::beans::Property& getProperty( const OUString& aName ) throw (css::beans::UnknownPropertyException );
     const css::beans::Property* hasProperty( const OUString& aName );
 
diff --git a/svx/source/table/tablecolumn.hxx b/svx/source/table/tablecolumn.hxx
index 8597e0676f70..5832ea02956c 100644
--- a/svx/source/table/tablecolumn.hxx
+++ b/svx/source/table/tablecolumn.hxx
@@ -41,6 +41,7 @@ public:
     virtual ~TableColumn() override;
 
     void dispose();
+    /// @throws css::uno::RuntimeException
     void throwIfDisposed() const throw (css::uno::RuntimeException);
 
     TableColumn& operator=( const TableColumn& );
diff --git a/svx/source/table/tablecolumns.hxx b/svx/source/table/tablecolumns.hxx
index fdc9973d1b30..56051fdc2113 100644
--- a/svx/source/table/tablecolumns.hxx
+++ b/svx/source/table/tablecolumns.hxx
@@ -35,6 +35,7 @@ public:
     virtual ~TableColumns() override;
 
     void dispose();
+    /// @throws css::uno::RuntimeException
     void throwIfDisposed() const throw (css::uno::RuntimeException);
 
     // XTableColumns
diff --git a/svx/source/table/tablemodel.hxx b/svx/source/table/tablemodel.hxx
index faafccd6d3fe..deb083498bd6 100644
--- a/svx/source/table/tablemodel.hxx
+++ b/svx/source/table/tablemodel.hxx
@@ -162,7 +162,9 @@ private:
     */
     virtual void SAL_CALL disposing() override;
 
+    /// @throws css::lang::IndexOutOfBoundsException
     TableRowRef getRow( sal_Int32 nRow ) const throw (css::lang::IndexOutOfBoundsException);
+    /// @throws css::lang::IndexOutOfBoundsException
     TableColumnRef getColumn( sal_Int32 nColumn ) const throw (css::lang::IndexOutOfBoundsException);
 
     void updateRows();
diff --git a/svx/source/table/tablerow.hxx b/svx/source/table/tablerow.hxx
index ba8df5bc75bf..bcdbe5ef9bc4 100644
--- a/svx/source/table/tablerow.hxx
+++ b/svx/source/table/tablerow.hxx
@@ -41,6 +41,7 @@ public:
     virtual ~TableRow() override;
 
     void dispose();
+    /// @throws css::uno::RuntimeException
     void throwIfDisposed() const throw (css::uno::RuntimeException);
 
     TableRow& operator=( const TableRow& );
diff --git a/svx/source/table/tablerows.hxx b/svx/source/table/tablerows.hxx
index 000d2a21f6cf..10788ff0afea 100644
--- a/svx/source/table/tablerows.hxx
+++ b/svx/source/table/tablerows.hxx
@@ -35,6 +35,7 @@ public:
     virtual ~TableRows() override;
 
     void dispose();
+    /// @throws css::uno::RuntimeException
     void throwIfDisposed() const throw (css::uno::RuntimeException);
 
     // XTableRows
diff --git a/svx/source/unodialogs/textconversiondlgs/chinese_translation_unodialog.hxx b/svx/source/unodialogs/textconversiondlgs/chinese_translation_unodialog.hxx
index ce4d2bb1a259..a65fc2e889de 100644
--- a/svx/source/unodialogs/textconversiondlgs/chinese_translation_unodialog.hxx
+++ b/svx/source/unodialogs/textconversiondlgs/chinese_translation_unodialog.hxx
@@ -79,6 +79,7 @@ public:
     static OUString getImplementationName_Static();
     static css::uno::Sequence< OUString > getSupportedServiceNames_Static();
 
+    /// @throws css::uno::Exception
     static css::uno::Reference< css::uno::XInterface > SAL_CALL
             create( css::uno::Reference< css::uno::XComponentContext > const & ) throw(css::uno::Exception)
     {
diff --git a/svx/source/unodraw/XPropertyTable.cxx b/svx/source/unodraw/XPropertyTable.cxx
index 2b1f57904915..60758c01d382 100644
--- a/svx/source/unodraw/XPropertyTable.cxx
+++ b/svx/source/unodraw/XPropertyTable.cxx
@@ -53,7 +53,10 @@ private:
 public:
     SvxUnoXPropertyTable( sal_Int16 nWhich, XPropertyList* pList ) throw();
 
+    /// @throws uno::RuntimeException
     virtual uno::Any getAny( const XPropertyEntry* pEntry ) const throw(uno::RuntimeException) = 0;
+    /// @throws uno::RuntimeException
+    /// @throws lang::IllegalArgumentException
     virtual std::unique_ptr<XPropertyEntry> createEntry(const OUString& rName, const uno::Any& rAny) const throw(uno::RuntimeException, lang::IllegalArgumentException) = 0;
 
     // XServiceInfo
diff --git a/svx/source/unodraw/unoprov.cxx b/svx/source/unodraw/unoprov.cxx
index 32beef82f295..71f911202dd5 100644
--- a/svx/source/unodraw/unoprov.cxx
+++ b/svx/source/unodraw/unoprov.cxx
@@ -1116,6 +1116,7 @@ bool SvxUnoGetResourceRanges( const short nWhich, int& nApiResIds, int& nIntResI
     return true;
 }
 
+/// @throws std::exception
 bool SvxUnoConvertResourceString( int nSourceResIds, int nDestResIds, int nCount, OUString& rString ) throw(std::exception)
 {
     // first, calculate the search string length without an optional number behind the name
@@ -1269,7 +1270,7 @@ static const sal_uInt16 SvxUnoColorNameResId[] =
     RID_SVXSTR_COLOR_TANGO_ALUMINIUM
 };
 
-
+/// @throws std::exception
 bool SvxUnoConvertResourceString( const sal_uInt16* pSourceResIds, const sal_uInt16* pDestResIds, int nCount, OUString& rString ) throw (std::exception)
 {
     //We replace e.g. "Gray 10%" with the translation of Gray, but we shouldn't