Review Request 121582: fix problems with longurl plugin

Eugene Shalygin eugene.shalygin at gmail.com
Thu Dec 18 17:12:53 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121582/
-----------------------------------------------------------

(Updated Dec. 18, 2014, 6:12 p.m.)


Review request for Choqok.


Changes
-------

That was stupid mistake of mine. I wrote that code for initial testing and forgot to rewrite it. So nice that you spotted it immidiately! Thank you!


Repository: choqok


Description
-------

Fix memleak and hopefully a race on unloading between destructor and parser in longurl plugin


Diffs (updated)
-----

  plugins/longurl/longurl.h da322c1 
  plugins/longurl/longurl.cpp 17bb175 

Diff: https://git.reviewboard.kde.org/r/121582/diff/


Testing
-------

No testing was done


Thanks,

Eugene Shalygin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/choqok-devel/attachments/20141218/2426aa2a/attachment.html>


More information about the Choqok-devel mailing list