Re: Review Request 126480: Fix assorted memory leaks and undefined accesses

2016-02-09 Thread Andrea Iacovitti
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126480/#review92199 --- It seems the diff can't be viewed nor downloaded. - A

Re: Review Request 127835: KHTML: Try again to fix Coverity memory leak in CSS background property

2016-05-21 Thread Andrea Iacovitti
2 = value2; } else { delete value2; } } return true; should be ok, but i don't know if it will make coverity happy. - Andrea Iacovitti On Mag. 18, 2016, 10:06 p.m., Michael Pyne wrote: > >

Re: Review Request 127835: KHTML: Try again to fix Coverity memory leak in CSS background property

2016-05-23 Thread Andrea Iacovitti
it is what i mean in term of code https://paste.kde.org/pf26dz1se (patch - tested). What do you think? - Andrea Iacovitti On Mag. 22, 2016, 11:53 p.m., Michael Pyne wrote: > > --- > This is an automatically generated e-mail. To reply,