Comment 21 for bug 1492153

Revision history for this message
su_v (suv-lp) wrote : Re: [Bug 1492153] Re: CDR import crashes after lib2geom update (rev >= 14226)

On 2016-02-14 18:56 (+0100), Eduard Braun wrote:
> I submitted the workaround in revision 14650 to fix the CDR import.

JFYI - on OS X 10.7.5, I get compiler warnings (form GCC 4.6.3 as well
as clang (based on LLVM 3.2svn)) about «unknown escape sequence '\s'».

--
<opinion>
Personally (as original reporter) I disagree with "fixes" which only
hide or disguise the real underlying issue [1], and don't help with
other possible occurrences of similar SVG content in third-party files
(for example files which have been converted to SVG externally (see
earlier comments 6-9 with sample SVG test cases) still trigger a crash
on open or import, with or without your "fix"). You can also easily
trigger the same kind of crash from within Inkscape trunk by editing a
path 'd' attribute in the XML Editor, (mis-)typing two consecutive 'Z'
path commands and hit the 'Set' button.

To me such kind of fixes might make sense if there's not real solution
available yet, or only e.g. within a recent major 2geom update in
unstable trunk, yet a new stable release is pending, and the "fix" would
be limited (as known and commented workaround) to the stable release
branch only.
</opinion>

[1] in your comment in the 2geom bug tracker (didn't lib2geom move to
github, btw?) you even explicitly lower the importance of the issue
(«not a high priority»).