
bulia byak schrieb:
It's better to use not g_string_append_printf but our own Inkscape::SVGOStringStream class - it has settable number precision and some other nice properties. You can find examples of its use in many places in the code.
Oh, and I just noticed that you used g_string_append_printf for creating your own path representation. In this case, it's simpler and better to use our existing path code for that. Just create a temporary NArtBpath, add the linetos, and then call sp_svg_write_path to get the string. This way all path writing is in one place, which is good for maintainability.
Thanks! With these suggestion the code gets _a lot_ cleaner. This is precisely what I was looking for, in the face of the ugly previous fix.:) The new version is in SVN now.
Cheers, Max