Kaydet (Commit) d7a8fa7a authored tarafından Stephan Bergmann's avatar Stephan Bergmann

Remove bogus assert

...that had been added with f0215dcc "unotools:
assert if TempFile::GetURL() fails due to missing file UCP".  But when SRCDIR is
a read-only tree, JunitTest_sc_unoapi_2 fails with that assert firing when
TempFile::GetURL is called from SfxMedium::CreateTempFile
(sfx2/source/doc/docfile.cxx), which in turn is prepared to handle an empty
return value from GetURL, and the test succeeds after removing the assert.  So
it looks like that assert was not precise enough to only fire in the scenarios
where it was intended to fire.

Change-Id: I333e9026ee14e9d1140b1b475793a5b15ededd70
Reviewed-on: https://gerrit.libreoffice.org/52085Tested-by: 's avatarJenkins <ci@libreoffice.org>
Reviewed-by: 's avatarStephan Bergmann <sbergman@redhat.com>
üst da4121ee
......@@ -409,7 +409,6 @@ OUString TempFile::GetFileName() const
OUString const & TempFile::GetURL() const
{
assert(!aName.isEmpty() && "TempFile::GetURL failed: unit test is leaking temp files, add the ucpfile1 component!");
return aName;
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment