At 28.07.2010 01:33, timothy b wrote:
With reference to bug 611078, https://bugzilla.gnome.org/show_bug.cgi?id=611078,
is this patch going to be incorporated into the next release or does it need to
be tested first?

It certainly needs to be tested before being incorporated in the next release. After a quick look it even needs to be modified before it can go in. The message_warning() may be useful for testing, if the right svg filter is used, but it does not belong into the production version.

But more seriously the direct use of g_snprintf() for floating point numbers is not correct. Please note the use of g_ascii_formatd() everywhere else in the file.

BTW: my minimal test would be the round-trip through the svg filter, i.e. exporting and importing e.g. render-test shall not deviate more form the original than it already does.

Regards,
        Hans

-------- Hans "at" Breuer "dot" Org -----------
Tell me what you need, and I'll tell you how to
get along without it.                -- Dilbert
_______________________________________________
dia-list mailing list
[email protected]
http://mail.gnome.org/mailman/listinfo/dia-list
FAQ at http://live.gnome.org/Dia/Faq
Main page at http://live.gnome.org/Dia

Reply via email to