editeng/source/accessibility/AccessibleEditableTextPara.cxx | 4 ++-- editeng/source/accessibility/AccessibleParaManager.cxx | 2 +- include/editeng/AccessibleEditableTextPara.hxx | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-)
New commits: commit 5348c79e012ed68810fe3ed229c2bc12e7c7ed53 Author: Michael Weghorn <[email protected]> AuthorDate: Thu Apr 3 12:23:09 2025 +0200 Commit: Michael Weghorn <[email protected]> CommitDate: Thu Apr 3 22:31:46 2025 +0200 editeng a11y: Simplify AccessibleEditableTextPara::FireEvent * Make method non-const, which avoid the need to const_cast. * Just use `this` directly instead of calling `getAccessibleContext`, which returns a reference to `this` again. This also prepeare for turning AccessibleEditableTextPara into an OAccessibleComponentHelper subclass in an upcoming commit. Change-Id: I74526e47070979ce34e89c165843ca3a5a9b1f60 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/183680 Tested-by: Jenkins Reviewed-by: Michael Weghorn <[email protected]> diff --git a/editeng/source/accessibility/AccessibleEditableTextPara.cxx b/editeng/source/accessibility/AccessibleEditableTextPara.cxx index c55e272b56af..e25fe7e5395b 100644 --- a/editeng/source/accessibility/AccessibleEditableTextPara.cxx +++ b/editeng/source/accessibility/AccessibleEditableTextPara.cxx @@ -536,9 +536,9 @@ void AccessibleEditableTextPara::SetEEOffset( const Point& rOffset ) maEEOffset = rOffset; } -void AccessibleEditableTextPara::FireEvent(const sal_Int16 nEventId, const uno::Any& rNewValue, const uno::Any& rOldValue) const +void AccessibleEditableTextPara::FireEvent(const sal_Int16 nEventId, const uno::Any& rNewValue, const uno::Any& rOldValue) { - uno::Reference < XAccessibleContext > xThis( const_cast< AccessibleEditableTextPara* > (this)->getAccessibleContext() ); + uno::Reference <XAccessibleContext> xThis(this); AccessibleEventObject aEvent(xThis, nEventId, rNewValue, rOldValue, -1); diff --git a/editeng/source/accessibility/AccessibleParaManager.cxx b/editeng/source/accessibility/AccessibleParaManager.cxx index f0be469d0249..649fe1585a33 100644 --- a/editeng/source/accessibility/AccessibleParaManager.cxx +++ b/editeng/source/accessibility/AccessibleParaManager.cxx @@ -311,7 +311,7 @@ public: mnEventId( nEventId ), mrNewValue( rNewValue ), mrOldValue( rOldValue ) {} - void operator()( ::accessibility::AccessibleEditableTextPara const & rPara ) + void operator()(::accessibility::AccessibleEditableTextPara& rPara) { rPara.FireEvent( mnEventId, mrNewValue, mrOldValue ); } diff --git a/include/editeng/AccessibleEditableTextPara.hxx b/include/editeng/AccessibleEditableTextPara.hxx index 9ed41a251b90..0be4bd1088d9 100644 --- a/include/editeng/AccessibleEditableTextPara.hxx +++ b/include/editeng/AccessibleEditableTextPara.hxx @@ -235,7 +235,7 @@ public: void Dispose(); /// Calls all Listener objects to tell them the change. Don't hold locks when calling this! - void FireEvent(const sal_Int16 nEventId, const css::uno::Any& rNewValue = css::uno::Any(), const css::uno::Any& rOldValue = css::uno::Any() ) const; + void FireEvent(const sal_Int16 nEventId, const css::uno::Any& rNewValue = css::uno::Any(), const css::uno::Any& rOldValue = css::uno::Any()); /// Sets the given state on the internal state set and fires STATE_CHANGE event. Don't hold locks when calling this! void SetState( const sal_Int64 nStateId );
