I just submitted a patch (id: 1896251) to the patch tracker that changes Inkscape::SVG::PathString to generate the shortest possible path string with decent spacing (that is, it uses the same spacing rules as the old PathString). It does this by eliminating repeated operators when it can, as well as using relative coordinates if it helps.
Using this patch I got file size reductions of about 10% on car.svg(z), the new about screen and some of my own files (I looked at both the uncompressed and the compressed svg's). To do this I opened the files, selected all objects and performed a translation of 0px on them, then saved the file (as uncompressed svg, and then again as compressed svg). The optimized path strings are about just as readable as the old ones (sometimes they're even easier to read).
I would welcome any suggestions/comments.
Jasper van de Gronde schrieb:
I just submitted a patch (id: 1896251) to the patch tracker [...]
Is that a valid LaunchPad id? Unfortunately, I can't seem to find the corresponding patch. :(
Max
P.S.: BTW, the "Patches" link on the main Inkscape webpage still points to the old sourceforge patch tracker. This should be changed to point to LaunchPad instead. (But I don't know who is responsible for the web page.)
Maximilian Albert wrote:
Jasper van de Gronde schrieb:
I just submitted a patch (id: 1896251) to the patch tracker [...]
Is that a valid LaunchPad id? Unfortunately, I can't seem to find the corresponding patch. :(
Max
P.S.: BTW, the "Patches" link on the main Inkscape webpage still points to the old sourceforge patch tracker. This should be changed to point to LaunchPad instead. (But I don't know who is responsible for the web page.)
No, it's in the SourceForge patch tracker, I couldn't find any patches area in LaunchPad (although I din't try terribly hard), and the Patches link indeed still points to the patch tracker.
Jasper van de Gronde wrote:
No, it's in the SourceForge patch tracker, I couldn't find any patches area in LaunchPad (although I din't try terribly hard), and the Patches link indeed still points to the patch tracker.
Sorry for my stupidity but I can't find it in SourceForge since the Inkscape project is closed down there. Do you have a URL at hand? As for LaunchPad, there is no separate patch tracker - you can simply open a new bug and attach the file to it (seems like this is an area of frequent confusion).
Max
Maximilian Albert wrote:
Jasper van de Gronde wrote:
No, it's in the SourceForge patch tracker, I couldn't find any patches area in LaunchPad (although I din't try terribly hard), and the Patches link indeed still points to the patch tracker.
Sorry for my stupidity but I can't find it in SourceForge since the Inkscape project is closed down there. Do you have a URL at hand? As for LaunchPad, there is no separate patch tracker - you can simply open a new bug and attach the file to it (seems like this is an area of frequent confusion).
I'm not surprised :) It feels really weird to submit a new/improved feature as a bug. But I've now submitted my patch as a bug to Launchpad (id: 193287).
participants (2)
-
Jasper van de Gronde -
Maximilian Albert