Review Request: Fix crashes due to using the same QUrls in multiple threads.
Sebastian Trueg
sebastian at trueg.de
Wed Jul 25 09:26:44 UTC 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105621/#review16358
-----------------------------------------------------------
Well, this is very unfortuanate. It even makes me think whether we should go back to the old version which returned QUrl::fromEncoded in every function... I only introduced the global static for performance reasons.
Anyway, I am fine with the patch if you add a big comment explaining the problem.
- Sebastian Trueg
On July 25, 2012, 8:58 a.m., David Faure wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105621/
> -----------------------------------------------------------
>
> (Updated July 25, 2012, 8:58 a.m.)
>
>
> Review request for Nepomuk and Release Team.
>
>
> Description
> -------
>
> Fix crashes due to using the same QUrls in multiple threads.
>
> I'm also working on a Qt4 QUrl fix (using QMutex), and this is already
> fixed in Qt5 (the rewrite does no more delayed parsing), but let's make
> our software stable as soon as possible, too, by using QThreadStorage
> to avoid the race.
>
>
> Diffs
> -----
>
> tools/onto2vocabularyclass.cpp 9a5a92b6210e695a6ad3cba9afe9aa1341759fc5
>
> Diff: http://git.reviewboard.kde.org/r/105621/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> David Faure
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/release-team/attachments/20120725/0f9be1e5/attachment.html>
More information about the release-team
mailing list