Hi,
I'm currently trying to implement customizable filters (try Filters>Experimental) with Ivan Louette, and found (thanks to ~suv) a small bug in the code of the feComponentTransfer primitive (see https://bugs.edge.launchpad.net/inkscape/+bug/638943).
To summarize, when the effect preview is updated (sometimes after many changes) the following error shows in the console: gboolean sp_document_add_resource(SPDocument*, const gchar*, SPObject*): assertion `!g_slist_find(rlist, object)' failed
sp_document_add_resource is called from componenttransfer.cpp, line 103, preceded by the comment: //do we need this? When this call is commented out, the error message doesn't show, but I'm not sure it's safe to remove it. It is also used in some other filters (pointlight.cpp, distantlight.cpp, spotlight.cpp and componenttransfer-funcnode.cpp) always with the same kind of comment (or // is this necessary?), but in these cases, the resources are released differently (compare sp_fedistantlight_release with sp_feComponentTransfer_release).
Could someone give a clue? or a fix ;) Regards, -- Nicolas
Is this still an issue in the cairo branch?
Cheers, Josh
On Thu, 2010-09-16 at 16:39 +0000, Nicolas Dufour wrote:
Hi,
I'm currently trying to implement customizable filters (try Filters>Experimental) with Ivan Louette, and found (thanks to ~suv) a small bug in the code of the feComponentTransfer primitive (see https://bugs.edge.launchpad.net/inkscape/+bug/638943).
To summarize, when the effect preview is updated (sometimes after many changes) the following error shows in the console: gboolean sp_document_add_resource(SPDocument*, const gchar*, SPObject*): assertion `!g_slist_find(rlist, object)' failed
sp_document_add_resource is called from componenttransfer.cpp, line 103, preceded by the comment: //do we need this? When this call is commented out, the error message doesn't show, but I'm not sure it's safe to remove it. It is also used in some other filters (pointlight.cpp, distantlight.cpp, spotlight.cpp and componenttransfer-funcnode.cpp) always with the same kind of comment (or // is this necessary?), but in these cases, the resources are released differently (compare sp_fedistantlight_release with sp_feComponentTransfer_release).
Could someone give a clue? or a fix ;) Regards, -- Nicolas
Start uncovering the many advantages of virtual appliances and start using them to simplify application deployment and accelerate your shift to cloud computing. http://p.sf.net/sfu/novell-sfdev2dev _______________________________________________ Inkscape-devel mailing list Inkscape-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/inkscape-devel
participants (3)
-
Joshua A. Andler -
Krzysztof Kosiński -
Nicolas Dufour