Comment 7 for bug 346681

Revision history for this message
Jacob M. Carson (reneviht) wrote :

Thanks for your interest in my code.

I would love to be able to rework my patch to make it viable for Inkscape. However...

First, there are two things you could mean by that. The first (and most likely) is that I should change the spiral gradient so that it simply implements Jasper's code directly. The second is to make Jasper's code the "backup rendering" somehow. This may be made clear in the mailing list; however, I have not subscribed to said list, and I couldn't find backissues in the places I looked.

Second, there's a lot more one could do with Jasper's code than what little I've done. If Jasper's code renders quickly, restricting that sort of thing to spiral gradients would be wasteful; it would make more sense to come up with a more general editor for fills of that type. If Jasper's code doesn't render quickly, then I'm not really interested in using it; the primary reason I use gradients instead of clipped/masked blurs is because the blur filters slow down my computer.

I would test it myself to see which hypothetical applies, but that leads to the third problem: When I click on the link Jasper provided, all I get is a page that says

Database Error: A database query syntax error has occurred. This may indicate a bug in the software. The last attempted database query was:

    (SQL query hidden)

from within function "MathRenderer::_recall". MySQL returned error "1267: Illegal mix of collations (latin1_swedish_ci,IMPLICIT) and (utf8_general_ci,COERCIBLE) for operation '=' (db3.osuosl.org)".

I saw the page once before, but I don't remember it well enough to implement Jasper's code.

On the off-chance you were waiting to hear from me before some part of the wrap-up, well, I won't be able to do anything in time. Best go ahead without me, and sorry I disappointed you.

Please forgive the lateness of my reply.