Re: [Inkscape-devel] Careful with C-strings!

On Mon, 2014-03-03 at 22:09 +0100, Johan Engelen wrote:

Another C-string induced memleak was added today. Actually, the change fixed one memleak, but replaces it with the same one. Fixed in r13101.

I'm going to rip out c-strings from uri handling. It's dangerous code and needs to be dealt with.

And then I'm going to make the std::string implimentation of path normalize.

I'll get that into trunk and merge the dom/ removal (which ironically has a uri implementation that uses std::string properly)

So, I expect the code might cause some issues, but I'm going to try and test as much as I can before the push. But please don't revert it, just post to me any errors or critical issues.

Best Regards, Martin Owens