[Nepomuk] Review Request: LocalSocketClient: Never delete the ClientConnection
Simeon Bird
bladud at gmail.com
Tue Dec 4 04:49:47 UTC 2012
> On Dec. 3, 2012, 4:36 a.m., Simeon Bird wrote:
> > This is an amazingly good idea.
> > The simpler this bit of code is, the more chance we have to debug some of the crazy crashes we're getting!
This should fix 305312 as well, right?
- Simeon
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107531/#review22926
-----------------------------------------------------------
On Nov. 30, 2012, 1:23 p.m., Vishesh Handa wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107531/
> -----------------------------------------------------------
>
> (Updated Nov. 30, 2012, 1:23 p.m.)
>
>
> Review request for Nepomuk, Soprano and Sebastian Trueg.
>
>
> Description
> -------
>
> LocalSocketClient: Never delete the ClientConnection
>
> The ClientConnection is not reference counted, and is passed to the the
> model and open iterators. Deleting it when has open iterators therefore
> leads to crashes.
>
> Instead, the ClientConnection is always persistent. Intead we provide
> functions to allow users of the ClientConnection to connect and
> disconnect.
>
> Also removed the concept of newConnection() from the ClientConnection.
> It no longer spawns a new socket for each thread, so there is no point
> of it existing. This new design is a lot simpler.
>
>
> Diffs
> -----
>
> client/clientconnection.h bfb3d4d
> client/clientconnection.cpp febd008
> client/localsocketclient.cpp f3579ff
>
> Diff: http://git.reviewboard.kde.org/r/107531/diff/
>
>
> Testing
> -------
>
> Yup. Everything works fine.
>
>
> Thanks,
>
> Vishesh Handa
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/nepomuk/attachments/20121204/3446c152/attachment-0001.html>
More information about the Nepomuk
mailing list