diff options
| author | David Mathog <mathog@caltech.edu> | 2014-03-25 20:14:20 +0000 |
|---|---|---|
| committer | Markus Engel <markus.engel@tum.de> | 2014-03-25 20:14:20 +0000 |
| commit | 2cadb5d282a9371129a28a142508120bd065454a (patch) | |
| tree | fde5e57816f0ea6e187c054571407e4bad3d9a88 /src/interface.cpp | |
| parent | fix crash introduced in rev. 13199. (revert the change to Glib::string (cras... (diff) | |
| download | inkscape-2cadb5d282a9371129a28a142508120bd065454a.tar.gz inkscape-2cadb5d282a9371129a28a142508120bd065454a.zip | |
Patch for several issues in libuemf.
-----
The patch:
1. Removes half a dozen or so code lines that didn't do anything.
2. Fixes a couple of missing tests for pointers which cannot be NULL. (Note that in many instances the pseudoobject functions in upmf.c do accept null pointers because the objects being constructed have optional parts.)
3. There was one "real" potential memory leak (line 6276 in upmf.c, numbers before the patch). I fixed the others too, but they just never happen (iconv would have to fail in some strange way.) There are no (real) memory access violations or memory leaks in any of this either in standalone libUEMF testing or in use as part of Inkscape as determined from valgrind runs.
(bzr r13211)
Diffstat (limited to 'src/interface.cpp')
0 files changed, 0 insertions, 0 deletions