[PATCH] fdo#46939: Crash after trying to rename autotext entry

[Tor Lillqvist]

Patch looks sane, but I still would like somebody else who is more
into C++ and our way of using C++ to say whether it's the Right Thing
to do or not.

(If all callers of a particular method catch the exceptions it can
throw even during fairly normal operation, and either just ignore
them, or turn them into returning some error-indicating return value
instead, what's the point in using the heavy-weight (?) and
harder-to-get-right exception mechanism in the first place? What am I
missing, except C++ evangelism?)

--tml