diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2023-05-31 15:31:36 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2023-05-31 18:37:34 +0200 |
| commit | 009b889616561176a230bc041699271697f95bf6 (patch) | |
| tree | e6c44c7b552f8a0764ce2e5a1afe9483dff5bcb4 /sot | |
| parent | 046e37faa295889157f0313f2300d93cb0f83b9e (diff) | |
do not throw DisposedException when inside a dispose() method
There is no need to do this, as the documentation of
css.lang.XComponent::dispose at
udkapi/com/sun/star/lang/XComponent.idl
states:
After this method has been called, the object should
behave as passive as possible, thus it should ignore all calls
Otherwise, the effect of throwing here is mostly to disturb the flow of
logic in caller code, preventing other parts of teardown from proceeding
smoothly.
Change-Id: I30e6d1b35f85b727debf4405a995fdc0a4fccde6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/152450
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sot')
| -rw-r--r-- | sot/source/unoolestorage/xolesimplestorage.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sot/source/unoolestorage/xolesimplestorage.cxx b/sot/source/unoolestorage/xolesimplestorage.cxx index d2cb93a04647..46b82b2b538e 100644 --- a/sot/source/unoolestorage/xolesimplestorage.cxx +++ b/sot/source/unoolestorage/xolesimplestorage.cxx @@ -546,7 +546,7 @@ void SAL_CALL OLESimpleStorage::dispose() std::unique_lock aGuard( m_aMutex ); if ( m_bDisposed ) - throw lang::DisposedException(); + return; if ( m_aListenersContainer.getLength(aGuard) ) { |