Memory Leak in the SVG parsing code


#1

Hi,

I found a small memory leak in the SVG parsing code.

In SVGState::parseText, parseShape is called for each each xml child, but its results is neither stored, nor deleted.
While waiting the support of SVG text, here is a simple fix:

Drawable* dp = parseShape (*e, path);
if (dp)
  delete dp;
...
Drawable* dp = parseText (*e);
if (dp)
  delete dp;

… or simply remove the forEachXmlChildElement block ?


#2

oh, thanks. I should finish writing that bit sometime!