On Oct 9, 2010, at 10:39 AM, Robin Palotai wrote:
I found a bug in the patch (1) which doesn't always dealloc the GList created (too strict boolean condition). The updated patch is attached.
Thanks. I'll take a look at them.
In general it is handy to attach patches to a pertinent Inkscape bug in launchpad[1], creating one if necessary. That's also a handy way to update individual ones with new patches.
But the main thing I wanted to mention was a question I had about source. You mention basing them on the current svn head, but we switched to Bazaar[2] a while back. Did you just misspeak about the source, or are these based on our older tree? (We do have some notes on getting started with it [3])
Thanks again for the work.
[1] https://bugs.launchpad.net/inkscape [2] https://code.launchpad.net/inkscape [3] http://wiki.inkscape.org/wiki/index.php/Working_with_Bazaar