diff options
| author | Michael Stahl <michael.stahl@allotropia.de> | 2023-12-13 19:49:23 +0100 |
|---|---|---|
| committer | Michael Stahl <michael.stahl@allotropia.de> | 2023-12-13 22:02:20 +0100 |
| commit | df23f570536c939ab4b44125ac8f62a8500e6f7c (patch) | |
| tree | d975f8d36790ca1198c7ec51517b8fbd1cd9d849 /sfx2 | |
| parent | 1956bd5f27fa10a336b9cf9e27751eca5ff907b5 (diff) | |
sfx2: remove wrong asserts
Mike Kaganski points out that "soffice -n anything.docx" triggers the
one in GetOutStream(), called from SfxMedium::CreateTempFile()
!bTransferSuccess path, so the assumption of which parts of SfxMedium
can be in use concurrently was wrong.
Change-Id: I40f321d71297cadcdf4bf9ba6f07c4db3d2ac523
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/160712
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
Diffstat (limited to 'sfx2')
| -rw-r--r-- | sfx2/source/doc/docfile.cxx | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/sfx2/source/doc/docfile.cxx b/sfx2/source/doc/docfile.cxx index 319615bacd20..adc22797f6c3 100644 --- a/sfx2/source/doc/docfile.cxx +++ b/sfx2/source/doc/docfile.cxx @@ -678,7 +678,6 @@ bool SfxMedium::IsSkipImages() const SvStream* SfxMedium::GetInStream() { - //assert(!pImpl->xStorage); // either SvStream or Storage if ( pImpl->m_pInStream ) return pImpl->m_pInStream.get(); @@ -749,7 +748,6 @@ void SfxMedium::CloseInStream_Impl(bool bInDestruction) SvStream* SfxMedium::GetOutStream() { - assert(!pImpl->xStorage); // either SvStream or Storage if ( !pImpl->m_pOutStream ) { // Create a temp. file if there is none because we always @@ -1760,7 +1758,6 @@ uno::Reference < embed::XStorage > SfxMedium::GetStorage( bool bCreateTempFile ) if ( pImpl->xStorage.is() || pImpl->m_bTriedStorage ) return pImpl->xStorage; - assert(!pImpl->m_pOutStream /*&& !pImpl->m_pInStream*/); // either SvStream or Storage uno::Sequence< uno::Any > aArgs( 2 ); auto pArgs = aArgs.getArray(); |