From 2bba0a173d5a7eb2fc41871e86bdc4a2a3d3ef8f Mon Sep 17 00:00:00 2001 From: Stephan Bergmann Date: Thu, 19 Jan 2017 17:47:30 +0100 Subject: [PATCH] New loplugin:dynexcspec: Add @throws documentation, vcl Change-Id: Icc398ec6cbb9152ee86b4ac0003619ae04206cd6 --- include/vcl/dndhelp.hxx | 6 ++++++ vcl/inc/canvasbitmap.hxx | 1 + vcl/inc/dndeventdispatcher.hxx | 6 ++++++ vcl/qa/cppunit/canvasbitmaptest.cxx | 1 + vcl/unx/gtk/a11y/atkaction.cxx | 1 + vcl/unx/gtk/a11y/atkcomponent.cxx | 1 + vcl/unx/gtk/a11y/atkeditabletext.cxx | 1 + vcl/unx/gtk/a11y/atkhypertext.cxx | 1 + vcl/unx/gtk/a11y/atkimage.cxx | 1 + vcl/unx/gtk/a11y/atkselection.cxx | 1 + vcl/unx/gtk/a11y/atktable.cxx | 1 + vcl/unx/gtk/a11y/atktext.cxx | 4 ++++ vcl/unx/gtk/a11y/atktextattributes.cxx | 1 + vcl/unx/gtk/a11y/atkutil.cxx | 14 ++++++++++++++ vcl/unx/gtk/a11y/atkvalue.cxx | 1 + vcl/unx/gtk/fpicker/SalGtkPicker.hxx | 4 ++++ vcl/unx/kde4/KDE4FilePicker.hxx | 23 +++++++++++++++++++++++ 17 files changed, 68 insertions(+) diff --git a/include/vcl/dndhelp.hxx b/include/vcl/dndhelp.hxx index babb5ad7e8ef..e9318a733273 100644 --- a/include/vcl/dndhelp.hxx +++ b/include/vcl/dndhelp.hxx @@ -56,25 +56,31 @@ public: virtual ~DragAndDropClient(); // css::datatransfer::dnd::XDragGestureListener + /// @throws css::uno::RuntimeException virtual void dragGestureRecognized( const css::datatransfer::dnd::DragGestureEvent& dge ) throw (css::uno::RuntimeException, std::exception); // css::datatransfer::dnd::XDragSourceListener + /// @throws css::uno::RuntimeException virtual void dragDropEnd(const css::datatransfer::dnd::DragSourceDropEvent& dsde ) throw (css::uno::RuntimeException, std::exception); // css::datatransfer::dnd::XDropTargetListener + /// @throws css::uno::RuntimeException virtual void drop( const css::datatransfer::dnd::DropTargetDropEvent& dtde ) throw (css::uno::RuntimeException, std::exception); + /// @throws css::uno::RuntimeException virtual void dragEnter( const css::datatransfer::dnd::DropTargetDragEnterEvent& dtdee ) throw (css::uno::RuntimeException, std::exception); + /// @throws css::uno::RuntimeException virtual void dragExit( const css::datatransfer::dnd::DropTargetEvent& dte ) throw (css::uno::RuntimeException, std::exception); + /// @throws css::uno::RuntimeException virtual void dragOver( const css::datatransfer::dnd::DropTargetDragEvent& dtde ) throw (css::uno::RuntimeException, std::exception); diff --git a/vcl/inc/canvasbitmap.hxx b/vcl/inc/canvasbitmap.hxx index 0b5dba96f6c7..5df5eb28a256 100644 --- a/vcl/inc/canvasbitmap.hxx +++ b/vcl/inc/canvasbitmap.hxx @@ -70,6 +70,7 @@ namespace unotools // XIntegerReadOnlyBitmap virtual css::uno::Sequence< ::sal_Int8 > SAL_CALL getData( css::rendering::IntegerBitmapLayout& bitmapLayout, const css::geometry::IntegerRectangle2D& rect ) throw (css::lang::IndexOutOfBoundsException, css::rendering::VolatileContentDestroyedException, css::uno::RuntimeException, std::exception) override; virtual css::uno::Sequence< ::sal_Int8 > SAL_CALL getPixel( css::rendering::IntegerBitmapLayout& bitmapLayout, const css::geometry::IntegerPoint2D& pos ) throw (css::lang::IndexOutOfBoundsException, css::rendering::VolatileContentDestroyedException, css::uno::RuntimeException, std::exception) override; + /// @throws css::uno::RuntimeException css::uno::Reference< css::rendering::XBitmapPalette > SAL_CALL getPalette( ) throw (css::uno::RuntimeException); virtual css::rendering::IntegerBitmapLayout SAL_CALL getMemoryLayout( ) throw (css::uno::RuntimeException, std::exception) override; diff --git a/vcl/inc/dndeventdispatcher.hxx b/vcl/inc/dndeventdispatcher.hxx index 79851892c144..4d35032c9cb2 100644 --- a/vcl/inc/dndeventdispatcher.hxx +++ b/vcl/inc/dndeventdispatcher.hxx @@ -46,22 +46,28 @@ class DNDEventDispatcher: public ::cppu::WeakImplHelper< * fire the events on the dnd listener container of the specified window */ + /// @throws css::uno::RuntimeException static sal_Int32 fireDragEnterEvent( vcl::Window *pWindow, const css::uno::Reference< css::datatransfer::dnd::XDropTargetDragContext >& xContext, const sal_Int8 nDropAction, const Point& rLocation, const sal_Int8 nSourceAction, const css::uno::Sequence< css::datatransfer::DataFlavor >& aFlavorList ) throw(css::uno::RuntimeException); + /// @throws css::uno::RuntimeException static sal_Int32 fireDragOverEvent( vcl::Window *pWindow, const css::uno::Reference< css::datatransfer::dnd::XDropTargetDragContext >& xContext, const sal_Int8 nDropAction, const Point& rLocation, const sal_Int8 nSourceAction ) throw(css::uno::RuntimeException); + /// @throws css::uno::RuntimeException static sal_Int32 fireDragExitEvent( vcl::Window *pWindow ) throw(css::uno::RuntimeException); + /// @throws css::uno::RuntimeException static sal_Int32 fireDropActionChangedEvent( vcl::Window *pWindow, const css::uno::Reference< css::datatransfer::dnd::XDropTargetDragContext >& xContext, const sal_Int8 nDropAction, const Point& rLocation, const sal_Int8 nSourceAction ) throw(css::uno::RuntimeException); + /// @throws css::uno::RuntimeException static sal_Int32 fireDropEvent( vcl::Window *pWindow, const css::uno::Reference< css::datatransfer::dnd::XDropTargetDropContext >& xContext, const sal_Int8 nDropAction, const Point& rLocation, const sal_Int8 nSourceAction, const css::uno::Reference< css::datatransfer::XTransferable >& xTransferable ) throw(css::uno::RuntimeException); + /// @throws css::uno::RuntimeException static sal_Int32 fireDragGestureEvent( vcl::Window *pWindow, const css::uno::Reference< css::datatransfer::dnd::XDragSource >& xSource, const css::uno::Any& event, const Point& rOrigin, const sal_Int8 nDragAction )throw(css::uno::RuntimeException); diff --git a/vcl/qa/cppunit/canvasbitmaptest.cxx b/vcl/qa/cppunit/canvasbitmaptest.cxx index 519973dcdcc5..9c9e5d85643b 100644 --- a/vcl/qa/cppunit/canvasbitmaptest.cxx +++ b/vcl/qa/cppunit/canvasbitmaptest.cxx @@ -315,6 +315,7 @@ private: return uno::Sequence< sal_Int8 >(); } + // @throws uno::RuntimeException uno::Reference< rendering::XBitmapPalette > SAL_CALL getPalette( ) throw (uno::RuntimeException) { uno::Reference< XBitmapPalette > aRet; diff --git a/vcl/unx/gtk/a11y/atkaction.cxx b/vcl/unx/gtk/a11y/atkaction.cxx index ef3c2ba4e9f1..3dc2fea35c72 100644 --- a/vcl/unx/gtk/a11y/atkaction.cxx +++ b/vcl/unx/gtk/a11y/atkaction.cxx @@ -43,6 +43,7 @@ getAsConst( const OString& rString ) return aUgly[ nIdx ].getStr(); } +/// @throws uno::RuntimeException static css::uno::Reference getAction( AtkAction *action ) throw (uno::RuntimeException) { diff --git a/vcl/unx/gtk/a11y/atkcomponent.cxx b/vcl/unx/gtk/a11y/atkcomponent.cxx index 60600b2e9ff0..84ab86b0c87a 100644 --- a/vcl/unx/gtk/a11y/atkcomponent.cxx +++ b/vcl/unx/gtk/a11y/atkcomponent.cxx @@ -23,6 +23,7 @@ using namespace ::com::sun::star; +/// @throws uno::RuntimeException static css::uno::Reference getComponent( AtkComponent *pComponent ) throw (uno::RuntimeException) { diff --git a/vcl/unx/gtk/a11y/atkeditabletext.cxx b/vcl/unx/gtk/a11y/atkeditabletext.cxx index ee984bdeb7b6..d7dda03a5321 100644 --- a/vcl/unx/gtk/a11y/atkeditabletext.cxx +++ b/vcl/unx/gtk/a11y/atkeditabletext.cxx @@ -27,6 +27,7 @@ using namespace ::com::sun::star; +/// @throws uno::RuntimeException static css::uno::Reference getEditableText( AtkEditableText *pEditableText ) throw (uno::RuntimeException) { diff --git a/vcl/unx/gtk/a11y/atkhypertext.cxx b/vcl/unx/gtk/a11y/atkhypertext.cxx index 4c28babf9ee2..4a10b3c82733 100644 --- a/vcl/unx/gtk/a11y/atkhypertext.cxx +++ b/vcl/unx/gtk/a11y/atkhypertext.cxx @@ -189,6 +189,7 @@ hyper_link_get_type() // ---------------------- AtkHyperText ---------------------- +/// @throws uno::RuntimeException static css::uno::Reference getHypertext( AtkHypertext *pHypertext ) throw (uno::RuntimeException) { diff --git a/vcl/unx/gtk/a11y/atkimage.cxx b/vcl/unx/gtk/a11y/atkimage.cxx index 873cddddbcd2..88897210fb2a 100644 --- a/vcl/unx/gtk/a11y/atkimage.cxx +++ b/vcl/unx/gtk/a11y/atkimage.cxx @@ -35,6 +35,7 @@ getAsConst( const OUString& rString ) return aUgly[ nIdx ].getStr(); } +/// @throws uno::RuntimeException static css::uno::Reference getImage( AtkImage *pImage ) throw (uno::RuntimeException) { diff --git a/vcl/unx/gtk/a11y/atkselection.cxx b/vcl/unx/gtk/a11y/atkselection.cxx index 62dd4289fab7..39bf065cbb37 100644 --- a/vcl/unx/gtk/a11y/atkselection.cxx +++ b/vcl/unx/gtk/a11y/atkselection.cxx @@ -23,6 +23,7 @@ using namespace ::com::sun::star; +/// @throws uno::RuntimeException static css::uno::Reference getSelection( AtkSelection *pSelection ) throw (uno::RuntimeException) { diff --git a/vcl/unx/gtk/a11y/atktable.cxx b/vcl/unx/gtk/a11y/atktable.cxx index 75dd5a11dbde..26fdef1659b2 100644 --- a/vcl/unx/gtk/a11y/atktable.cxx +++ b/vcl/unx/gtk/a11y/atktable.cxx @@ -48,6 +48,7 @@ getAsConst( const OUString& rString ) /*****************************************************************************/ +/// @throws uno::RuntimeException static css::uno::Reference getTable( AtkTable *pTable ) throw (uno::RuntimeException) { diff --git a/vcl/unx/gtk/a11y/atktext.cxx b/vcl/unx/gtk/a11y/atktext.cxx index fbfd0fb41e11..b08edd1fa18f 100644 --- a/vcl/unx/gtk/a11y/atktext.cxx +++ b/vcl/unx/gtk/a11y/atktext.cxx @@ -133,6 +133,7 @@ adjust_boundaries( css::uno::Reference cons /*****************************************************************************/ +/// @throws uno::RuntimeException static css::uno::Reference getText( AtkText *pText ) throw (uno::RuntimeException) { @@ -149,6 +150,7 @@ static css::uno::Reference /*****************************************************************************/ +/// @throws uno::RuntimeException static css::uno::Reference getTextMarkup( AtkText *pText ) throw (uno::RuntimeException) { @@ -165,6 +167,7 @@ static css::uno::Reference /*****************************************************************************/ +/// @throws uno::RuntimeException static css::uno::Reference getTextAttributes( AtkText *pText ) throw (uno::RuntimeException) { @@ -181,6 +184,7 @@ static css::uno::Reference /*****************************************************************************/ +/// @throws uno::RuntimeException static css::uno::Reference getMultiLineText( AtkText *pText ) throw (uno::RuntimeException) { diff --git a/vcl/unx/gtk/a11y/atktextattributes.cxx b/vcl/unx/gtk/a11y/atktextattributes.cxx index f9d4bb26674f..4fa1afb42266 100644 --- a/vcl/unx/gtk/a11y/atktextattributes.cxx +++ b/vcl/unx/gtk/a11y/atktextattributes.cxx @@ -207,6 +207,7 @@ String2Float( uno::Any& rAny, const gchar * value ) /*****************************************************************************/ +/// @throws uno::RuntimeException static css::uno::Reference getComponent( AtkText *pText ) throw (uno::RuntimeException) { diff --git a/vcl/unx/gtk/a11y/atkutil.cxx b/vcl/unx/gtk/a11y/atkutil.cxx index 4a6db3dfcbd3..da165245be0b 100644 --- a/vcl/unx/gtk/a11y/atkutil.cxx +++ b/vcl/unx/gtk/a11y/atkutil.cxx @@ -139,36 +139,50 @@ class DocumentFocusListener : std::set< uno::Reference< uno::XInterface > > m_aRefList; public: + /// @trhows lang::IndexOutOfBoundsException + /// @throws uno::RuntimeException void attachRecursive( const uno::Reference< accessibility::XAccessible >& xAccessible ) throw (lang::IndexOutOfBoundsException, uno::RuntimeException); + /// @trhows lang::IndexOutOfBoundsException + /// @throws uno::RuntimeException void attachRecursive( const uno::Reference< accessibility::XAccessible >& xAccessible, const uno::Reference< accessibility::XAccessibleContext >& xContext ) throw (lang::IndexOutOfBoundsException, uno::RuntimeException); + /// @trhows lang::IndexOutOfBoundsException + /// @throws uno::RuntimeException void attachRecursive( const uno::Reference< accessibility::XAccessible >& xAccessible, const uno::Reference< accessibility::XAccessibleContext >& xContext, const uno::Reference< accessibility::XAccessibleStateSet >& xStateSet ) throw (lang::IndexOutOfBoundsException, uno::RuntimeException); + /// @trhows lang::IndexOutOfBoundsException + /// @throws uno::RuntimeException void detachRecursive( const uno::Reference< accessibility::XAccessible >& xAccessible ) throw (lang::IndexOutOfBoundsException, uno::RuntimeException); + /// @trhows lang::IndexOutOfBoundsException + /// @throws uno::RuntimeException void detachRecursive( const uno::Reference< accessibility::XAccessible >& xAccessible, const uno::Reference< accessibility::XAccessibleContext >& xContext ) throw (lang::IndexOutOfBoundsException, uno::RuntimeException); + /// @trhows lang::IndexOutOfBoundsException + /// @throws uno::RuntimeException void detachRecursive( const uno::Reference< accessibility::XAccessible >& xAccessible, const uno::Reference< accessibility::XAccessibleContext >& xContext, const uno::Reference< accessibility::XAccessibleStateSet >& xStateSet ) throw (lang::IndexOutOfBoundsException, uno::RuntimeException); + /// @trhows lang::IndexOutOfBoundsException + /// @throws uno::RuntimeException static uno::Reference< accessibility::XAccessible > getAccessible(const lang::EventObject& aEvent ) throw (lang::IndexOutOfBoundsException, uno::RuntimeException); diff --git a/vcl/unx/gtk/a11y/atkvalue.cxx b/vcl/unx/gtk/a11y/atkvalue.cxx index def434be6e75..067333af5916 100644 --- a/vcl/unx/gtk/a11y/atkvalue.cxx +++ b/vcl/unx/gtk/a11y/atkvalue.cxx @@ -25,6 +25,7 @@ using namespace ::com::sun::star; +/// @throws uno::RuntimeException static css::uno::Reference getValue( AtkValue *pValue ) throw (uno::RuntimeException) { diff --git a/vcl/unx/gtk/fpicker/SalGtkPicker.hxx b/vcl/unx/gtk/fpicker/SalGtkPicker.hxx index abee669c2c8b..74c46af9c995 100644 --- a/vcl/unx/gtk/fpicker/SalGtkPicker.hxx +++ b/vcl/unx/gtk/fpicker/SalGtkPicker.hxx @@ -49,12 +49,16 @@ class SalGtkPicker osl::Mutex m_rbHelperMtx; GtkWidget *m_pDialog; protected: + /// @throws css::uno::RuntimeException void SAL_CALL implsetTitle( const OUString& aTitle ) throw( css::uno::RuntimeException ); + /// @throws css::lang::IllegalArgumentException + /// @throws css::uno::RuntimeException void SAL_CALL implsetDisplayDirectory( const OUString& rDirectory ) throw( css::lang::IllegalArgumentException, css::uno::RuntimeException ); + /// @throws css::uno::RuntimeException OUString SAL_CALL implgetDisplayDirectory( ) throw( css::uno::RuntimeException ); OUString uritounicode(const gchar *pIn); diff --git a/vcl/unx/kde4/KDE4FilePicker.hxx b/vcl/unx/kde4/KDE4FilePicker.hxx index 129e7a702aef..9720e766a404 100644 --- a/vcl/unx/kde4/KDE4FilePicker.hxx +++ b/vcl/unx/kde4/KDE4FilePicker.hxx @@ -134,6 +134,7 @@ public: virtual void SAL_CALL cancel( ) throw( css::uno::RuntimeException, std::exception ) override; // XEventListener + /// @throws css::uno::RuntimeException virtual void SAL_CALL disposing( const css::lang::EventObject &rEvent ) throw( css::uno::RuntimeException ); using cppu::WeakComponentImplHelperBase::disposing; @@ -144,35 +145,57 @@ public: private Q_SLOTS: // XExecutableDialog functions + /// @throws css::uno::RuntimeException void setTitleSlot( const OUString &rTitle ) throw( css::uno::RuntimeException ) { return setTitle( rTitle ); } + /// @throws css::uno::RuntimeException sal_Int16 executeSlot() throw( css::uno::RuntimeException ) { return execute(); } // XFilePicker functions + /// @throws css::uno::RuntimeException void setMultiSelectionModeSlot( bool bMode ) throw( css::uno::RuntimeException ) { return setMultiSelectionMode( bMode ); } + /// @throws css::uno::RuntimeException void setDefaultNameSlot( const OUString &rName ) throw( css::uno::RuntimeException ) { return setDefaultName( rName ); } + /// @throws css::uno::RuntimeException void setDisplayDirectorySlot( const OUString &rDirectory ) throw( css::uno::RuntimeException ) { return setDisplayDirectory( rDirectory ); } + /// @throws css::uno::RuntimeException OUString getDisplayDirectorySlot() throw( css::uno::RuntimeException ) { return getDisplayDirectory(); } + /// @throws css::uno::RuntimeException css::uno::Sequence< OUString > getFilesSlot() throw( css::uno::RuntimeException ) { return getFiles(); } // XFilterManager functions + /// @throws css::lang::IllegalArgumentException + /// @throws css::uno::RuntimeException void appendFilterSlot( const OUString &rTitle, const OUString &rFilter ) throw( css::lang::IllegalArgumentException, css::uno::RuntimeException ) { return appendFilter( rTitle, rFilter ); } + /// @throws css::lang::IllegalArgumentException + /// @throws css::uno::RuntimeException void setCurrentFilterSlot( const OUString &rTitle ) throw( css::lang::IllegalArgumentException, css::uno::RuntimeException ) { return setCurrentFilter( rTitle ); } + /// @throws css::uno::RuntimeException OUString getCurrentFilterSlot() throw( css::uno::RuntimeException ) { return getCurrentFilter(); } // XFilterGroupManager functions + /// @throws css::lang::IllegalArgumentException + /// @throws css::uno::RuntimeException void appendFilterGroupSlot( const OUString &rGroupTitle, const css::uno::Sequence< css::beans::StringPair > &rFilters ) throw (css::lang::IllegalArgumentException, css::uno::RuntimeException) { return appendFilterGroup( rGroupTitle, rFilters ); } // XFilePickerControlAccess functions + /// @throws css::uno::RuntimeException void setValueSlot( sal_Int16 nControlId, sal_Int16 nControlAction, const css::uno::Any &rValue ) throw (css::uno::RuntimeException) { return setValue( nControlId, nControlAction, rValue ); } + /// @throws css::uno::RuntimeException css::uno::Any getValueSlot( sal_Int16 nControlId, sal_Int16 nControlAction ) throw (css::uno::RuntimeException) { return getValue( nControlId, nControlAction ); } + /// @throws css::uno::RuntimeException void enableControlSlot( sal_Int16 nControlId, bool bEnable ) throw( css::uno::RuntimeException ) { return enableControl( nControlId, bEnable ); } + /// @throws css::uno::RuntimeException void setLabelSlot( sal_Int16 nControlId, const OUString &rLabel ) throw (css::uno::RuntimeException) { return setLabel( nControlId, rLabel ); } + /// @throws css::uno::RuntimeException OUString getLabelSlot( sal_Int16 nControlId ) throw (css::uno::RuntimeException) { return getLabel( nControlId ); } // XFilePicker2 functions + /// @throws css::uno::RuntimeException css::uno::Sequence< OUString > getSelectedFilesSlot() throw (css::uno::RuntimeException) { return getSelectedFiles(); } // XInitialization + /// @throws css::uno::Exception + /// @throws css::uno::RuntimeException void initializeSlot( const css::uno::Sequence< css::uno::Any > &rArguments ) throw( css::uno::Exception, css::uno::RuntimeException ) { return initialize( rArguments ); } Q_SIGNALS: