10 Jun
2014
10 Jun
'14
5:35 p.m.
On 10-6-2014 17:29, Guiu Rocafort wrote:
Hi !
Making a simple check to see if all the new operators are freed using delete i found that there are some memory leaks in main.cpp, in the function replaceArgs ( which only affects the windows version of inkscape ).
I've added the missing delete operators and those memory leaks should be fixed. I attach the .patch file, if you want me to fill a bug report and then submit the patch instead please tell me.
Did you test your patch? While you are right that there are some memleaks, there are less than you think there are :) (e.g. your "delete[] block;" on line 1984 is wrong)
I'll rework and commit it.
regards, Johan