Comment 20 for bug 174720

Revision history for this message
Jaspervdg (jaspervdg) wrote : Re: [Bug 174720] Re: currentColor in style= is misinterpreted

bulia byak wrote:
> On Tue, Apr 8, 2008 at 7:56 AM, Martin von Gagern
> <email address hidden> wrote:
> ...
>> Another interesting way to avoid these issues might be some more
>> extensive verification. An invalid style attribute like the one above
>> certainly should never have been generated by inkscape itself. If you
>> were to implement appropriate error checking code in sp_style_read_*,
>> you could enable it while processing internally generated attributes.
>> You might wish to disable it for file reading or attributes entered in
>> the XML editor, or even better, you might wish to handle errors there
>> differently. So perhaps you want a format error listener interface, and
>> pass instance of that along. Quite a bit of work, but maybe worthwhile,
>> as it would also help identify SVG documents in the wild that aren't
>> fully understood by inkscape.
>
> I don't think we want to overburden Inkscape itself with checks over
> and above those prescribed by the SVG/CSS standard. But an external
> consistency checker for Inkscape-produced files might be a great
> addition to the testing project.

Instead of making sure that Inkscape knows how to parse/serialize CSS
properly, wouldn't it be better to simply use libcroco for all of this?
At first glance it seems to provide plenty of functionality.