[PUSHED][PATCH] fdo#46939: Crash after trying to rename autotext entry
Caolán McNamara
caolanm at redhat.com
Thu Mar 8 06:08:36 PST 2012
On Wed, 2012-03-07 at 04:20 +0100, Dézsi Szabolcs wrote:
>
> xRoot->renameElement ( aOldStreamName, aNewStreamName );
>
> It can be seen a few lines later, that xBlkRoot is handled the same way,
> catching a container::ElementExistException, but doing nothing in the catch
> block.
Yeah, that logic suggests that if we catch in one place, we should catch
in the other, so PUSHED.
Though I see the reason an exception is thrown is because the oldname
and the newname are the same, i.e. the rename doesn't make sense, it
already has that name. So I stuck the offending block inside the
existing new != old test.
C.
More information about the LibreOffice
mailing list