20 Apr
2009
20 Apr
'09
3:53 p.m.
Preben Soeberg wrote:
I have posted a patch for https://bugs.launchpad.net/inkscape/+bug/193926 The patch fixes the divisor to be calculated automatically by default.
Could you take a look at it and commit it?
I've had a look and in principle the fix to the sp-feconvolvematrix code seems fine. But:
- Could you perhaps base the fix on a more recent version? sp-feconvolvematrix.cpp no longer exists (it is now src/filters/convolvematrix.cpp).
- I have hardly any experience with the UI code, so if someone else could take a look at that part that would be great. (Otherwise, if you'll update the patch I'll just try it and commit it if it seems to work.)
Thanks for your efforts.