sc/source/ui/Accessibility/AccessibleCsvControl.cxx | 74 +------------------- sc/source/ui/inc/AccessibleCsvControl.hxx | 53 +++----------- 2 files changed, 18 insertions(+), 109 deletions(-)
New commits: commit 28b7b783202d251f43658b2ff1d8df3f10d6d1b8 Author: Stephan Bergmann <sberg...@redhat.com> AuthorDate: Fri Jan 27 15:30:06 2023 +0100 Commit: Stephan Bergmann <sberg...@redhat.com> CommitDate: Sat Jan 28 17:43:29 2023 +0000 Use ImplInheritanceHelper in ScAccessibleCsvCell Change-Id: I46e9985fe3bb05dc677b4046b3cad18dea1086b9 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/146276 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sberg...@redhat.com> diff --git a/sc/source/ui/Accessibility/AccessibleCsvControl.cxx b/sc/source/ui/Accessibility/AccessibleCsvControl.cxx index 075657fd9b21..eeaee6e2f91c 100644 --- a/sc/source/ui/Accessibility/AccessibleCsvControl.cxx +++ b/sc/source/ui/Accessibility/AccessibleCsvControl.cxx @@ -1219,7 +1219,7 @@ ScAccessibleCsvCell::ScAccessibleCsvCell( ScCsvGrid& rGrid, OUString aCellText, sal_Int32 nRow, sal_Int32 nColumn ) : - ScAccessibleCsvControl( rGrid ), + ImplInheritanceHelper( rGrid ), AccessibleStaticTextBase( SvxEditSourcePtr() ), maCellText(std::move( aCellText )), mnLine( nRow ? (nRow + rGrid.GetFirstVisLine() - 1) : CSV_LINE_HEADER ), @@ -1310,11 +1310,11 @@ sal_Int64 SAL_CALL ScAccessibleCsvCell::getAccessibleStateSet() // XInterface ----------------------------------------------------------------- -IMPLEMENT_FORWARD_XINTERFACE2( ScAccessibleCsvCell, ScAccessibleCsvControl, AccessibleStaticTextBase ) +IMPLEMENT_FORWARD_XINTERFACE2( ScAccessibleCsvCell, ImplInheritanceHelper, AccessibleStaticTextBase ) // XTypeProvider -------------------------------------------------------------- -IMPLEMENT_FORWARD_XTYPEPROVIDER2( ScAccessibleCsvCell, ScAccessibleCsvControl, AccessibleStaticTextBase ) +IMPLEMENT_FORWARD_XTYPEPROVIDER2( ScAccessibleCsvCell, ImplInheritanceHelper, AccessibleStaticTextBase ) // helpers -------------------------------------------------------------------- diff --git a/sc/source/ui/inc/AccessibleCsvControl.hxx b/sc/source/ui/inc/AccessibleCsvControl.hxx index ea3308fc4eb5..cb1efc309353 100644 --- a/sc/source/ui/inc/AccessibleCsvControl.hxx +++ b/sc/source/ui/inc/AccessibleCsvControl.hxx @@ -30,7 +30,6 @@ #include <rtl/ustrbuf.hxx> #include <comphelper/accessiblecomponenthelper.hxx> #include <cppuhelper/implbase.hxx> -#include <cppuhelper/implbase1.hxx> #include <editeng/AccessibleStaticTextBase.hxx> #include <comphelper/uno3.hxx> #include <map> @@ -400,8 +399,9 @@ private: }; /** Accessible class representing a cell of the CSV grid control. */ -class ScAccessibleCsvCell : public ScAccessibleCsvControl - , public cppu::ImplHelper1<css::accessibility::XAccessible> +class ScAccessibleCsvCell : public cppu::ImplInheritanceHelper< + ScAccessibleCsvControl, + css::accessibility::XAccessible> , public ::accessibility::AccessibleStaticTextBase { protected: commit ec68d4bb894882835aed28cee8537481afd0e302 Author: Stephan Bergmann <sberg...@redhat.com> AuthorDate: Fri Jan 27 15:25:24 2023 +0100 Commit: Stephan Bergmann <sberg...@redhat.com> CommitDate: Sat Jan 28 17:43:22 2023 +0000 Use ImplInheritanceHelper in ScAccessibleCsvGrid Change-Id: I1713fb24ed16ef9db6d2815a286d4182f19a5e6e Reviewed-on: https://gerrit.libreoffice.org/c/core/+/146275 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sberg...@redhat.com> diff --git a/sc/source/ui/Accessibility/AccessibleCsvControl.cxx b/sc/source/ui/Accessibility/AccessibleCsvControl.cxx index bc5dd8208b80..075657fd9b21 100644 --- a/sc/source/ui/Accessibility/AccessibleCsvControl.cxx +++ b/sc/source/ui/Accessibility/AccessibleCsvControl.cxx @@ -727,7 +727,7 @@ static sal_uInt32 lcl_GetGridColumn( sal_Int32 nApiColumn ) } ScAccessibleCsvGrid::ScAccessibleCsvGrid(ScCsvGrid& rGrid) - : ScAccessibleCsvControl(rGrid) + : ImplInheritanceHelper(rGrid) { } @@ -1069,39 +1069,6 @@ void SAL_CALL ScAccessibleCsvGrid::deselectAccessibleChild( sal_Int64 nSelectedC implSelectColumn( nColumn, false ); } -// XInterface ----------------------------------------------------------------- - -Any SAL_CALL ScAccessibleCsvGrid::queryInterface( const css::uno::Type& rType ) -{ - Any aAny( ScAccessibleCsvGridImpl::queryInterface( rType ) ); - return aAny.hasValue() ? aAny : ScAccessibleCsvControl::queryInterface( rType ); -} - -void SAL_CALL ScAccessibleCsvGrid::acquire() noexcept -{ - ScAccessibleCsvControl::acquire(); -} - -void SAL_CALL ScAccessibleCsvGrid::release() noexcept -{ - ScAccessibleCsvControl::release(); -} - -// XTypeProvider -------------------------------------------------------------- - -Sequence< css::uno::Type > SAL_CALL ScAccessibleCsvGrid::getTypes() -{ - return ::comphelper::concatSequences( ScAccessibleCsvControl::getTypes(), - Sequence { - cppu::UnoType<XAccessibleTable>::get(), - cppu::UnoType<XAccessibleSelection>::get() }); -} - -Sequence< sal_Int8 > SAL_CALL ScAccessibleCsvGrid::getImplementationId() -{ - return css::uno::Sequence<sal_Int8>(); -} - // events --------------------------------------------------------------------- void ScAccessibleCsvGrid::SendFocusEvent( bool bFocused ) diff --git a/sc/source/ui/inc/AccessibleCsvControl.hxx b/sc/source/ui/inc/AccessibleCsvControl.hxx index 1c228a1c9317..ea3308fc4eb5 100644 --- a/sc/source/ui/inc/AccessibleCsvControl.hxx +++ b/sc/source/ui/inc/AccessibleCsvControl.hxx @@ -31,7 +31,6 @@ #include <comphelper/accessiblecomponenthelper.hxx> #include <cppuhelper/implbase.hxx> #include <cppuhelper/implbase1.hxx> -#include <cppuhelper/implbase3.hxx> #include <editeng/AccessibleStaticTextBase.hxx> #include <comphelper/uno3.hxx> #include <map> @@ -214,14 +213,12 @@ private: class ScCsvGrid; class ScAccessibleCsvCell; -typedef ::cppu::ImplHelper3< - css::accessibility::XAccessible, - css::accessibility::XAccessibleTable, - css::accessibility::XAccessibleSelection > - ScAccessibleCsvGridImpl; - /** Accessible class representing the CSV grid control. */ -class ScAccessibleCsvGrid : public ScAccessibleCsvControl, public ScAccessibleCsvGridImpl +class ScAccessibleCsvGrid : public cppu::ImplInheritanceHelper< + ScAccessibleCsvControl, + css::accessibility::XAccessible, + css::accessibility::XAccessibleTable, + css::accessibility::XAccessibleSelection> { protected: typedef std::map< sal_Int64, rtl::Reference<ScAccessibleCsvCell> > XAccessibleSet; @@ -349,22 +346,6 @@ public: /** Deselects the child with the specified index in all selected children. */ virtual void SAL_CALL deselectAccessibleChild( sal_Int64 nSelectedChildIndex ) override; - // XInterface ------------------------------------------------------------- - - virtual css::uno::Any SAL_CALL queryInterface( const css::uno::Type& rType ) override; - - virtual void SAL_CALL acquire() noexcept override; - - virtual void SAL_CALL release() noexcept override; - - // XTypeProvider ---------------------------------------------------------- - - /** Returns a sequence with all supported interface types. */ - virtual css::uno::Sequence< css::uno::Type > SAL_CALL getTypes() override; - - /** Returns an implementation ID. */ - virtual css::uno::Sequence< sal_Int8 > SAL_CALL getImplementationId() override; - // events ----------------------------------------------------------------- public: /** Sends a GetFocus or LoseFocus event to all listeners. */ commit 154035563590181d096d8eccf0f549f314ddecde Author: Stephan Bergmann <sberg...@redhat.com> AuthorDate: Fri Jan 27 15:20:31 2023 +0100 Commit: Stephan Bergmann <sberg...@redhat.com> CommitDate: Sat Jan 28 17:43:14 2023 +0000 Use ImplInheritanceHelper in ScAccessibleCsvRuler Change-Id: I24b64a97e5431d1d25d2894c6e2ae41ec13245c4 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/146274 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sberg...@redhat.com> diff --git a/sc/source/ui/Accessibility/AccessibleCsvControl.cxx b/sc/source/ui/Accessibility/AccessibleCsvControl.cxx index 9d7e2bab5799..bc5dd8208b80 100644 --- a/sc/source/ui/Accessibility/AccessibleCsvControl.cxx +++ b/sc/source/ui/Accessibility/AccessibleCsvControl.cxx @@ -246,7 +246,7 @@ static void lcl_FillFontAttributes( Sequence< PropertyValue >& rSeq, const vcl:: } ScAccessibleCsvRuler::ScAccessibleCsvRuler(ScCsvRuler& rRuler) - : ScAccessibleCsvControl(rRuler) + : ImplInheritanceHelper(rRuler) { constructStringBuffer(); } @@ -611,37 +611,6 @@ sal_Bool SAL_CALL ScAccessibleCsvRuler::scrollSubstringTo( sal_Int32 /* nStartIn return false; } -// XInterface ----------------------------------------------------------------- - -Any SAL_CALL ScAccessibleCsvRuler::queryInterface( const css::uno::Type& rType ) -{ - Any aAny( ScAccessibleCsvRulerImpl::queryInterface( rType ) ); - return aAny.hasValue() ? aAny : ScAccessibleCsvControl::queryInterface( rType ); -} - -void SAL_CALL ScAccessibleCsvRuler::acquire() noexcept -{ - ScAccessibleCsvControl::acquire(); -} - -void SAL_CALL ScAccessibleCsvRuler::release() noexcept -{ - ScAccessibleCsvControl::release(); -} - -// XTypeProvider -------------------------------------------------------------- - -Sequence< css::uno::Type > SAL_CALL ScAccessibleCsvRuler::getTypes() -{ - return ::comphelper::concatSequences( ScAccessibleCsvControl::getTypes(), - Sequence { cppu::UnoType<XAccessibleText>::get() }); -} - -Sequence< sal_Int8 > SAL_CALL ScAccessibleCsvRuler::getImplementationId() -{ - return css::uno::Sequence<sal_Int8>(); -} - // events --------------------------------------------------------------------- void ScAccessibleCsvRuler::SendCaretEvent() diff --git a/sc/source/ui/inc/AccessibleCsvControl.hxx b/sc/source/ui/inc/AccessibleCsvControl.hxx index 6dfc22433880..1c228a1c9317 100644 --- a/sc/source/ui/inc/AccessibleCsvControl.hxx +++ b/sc/source/ui/inc/AccessibleCsvControl.hxx @@ -29,8 +29,8 @@ #include <rtl/ref.hxx> #include <rtl/ustrbuf.hxx> #include <comphelper/accessiblecomponenthelper.hxx> +#include <cppuhelper/implbase.hxx> #include <cppuhelper/implbase1.hxx> -#include <cppuhelper/implbase2.hxx> #include <cppuhelper/implbase3.hxx> #include <editeng/AccessibleStaticTextBase.hxx> #include <comphelper/uno3.hxx> @@ -83,11 +83,11 @@ protected: class ScCsvRuler; -typedef ::cppu::ImplHelper2<css::accessibility::XAccessible, - css::accessibility::XAccessibleText> ScAccessibleCsvRulerImpl; - /** Accessible class representing the CSV ruler control. */ -class ScAccessibleCsvRuler : public ScAccessibleCsvControl, public ScAccessibleCsvRulerImpl +class ScAccessibleCsvRuler : public cppu::ImplInheritanceHelper< + ScAccessibleCsvControl, + css::accessibility::XAccessible, + css::accessibility::XAccessibleText> { private: OUStringBuffer maBuffer; /// Contains the text representation of the ruler. @@ -175,14 +175,6 @@ public: virtual sal_Bool SAL_CALL scrollSubstringTo( sal_Int32 nStartIndex, sal_Int32 nEndIndex, css::accessibility::AccessibleScrollType aScrollType) override; - // XInterface ------------------------------------------------------------- - - DECLARE_XINTERFACE() - - // XTypeProvider ---------------------------------------------------------- - - DECLARE_XTYPEPROVIDER() - // events ----------------------------------------------------------------- public: /** Sends a caret changed event to all listeners. */