Use isEmpty and endsWith

The isEmpty could be redundant, since the first test for length in
the old code seemed to be just a pre-condition for evaluating the
expressions in the second one.

Change-Id: I0d1838d5eb143196a0d7be0cbadf9f0a59cff71b
Simplify by choosing shape kind in SvXMLElementExport constructor

Change-Id: I6d8e1e3c78675b6096f354cace3ff060801cad9e
Remove stray fprintf

Change-Id: I9d85108b8c2659f1b167299840772f51f80b2e29