> On Aug. 15, 2011, 4:34 a.m., Commit Hook wrote: > > This review has been submitted with commit > > caece88d2c5b5b6f0da3dc048878831814aab9f3 by Jekyll Wu to branch master.
The above commit message is invalid. It was intended for review 102239 (off by one). - Dawit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102238/#review5700 ----------------------------------------------------------- On Aug. 12, 2011, 3:45 a.m., Dawit Alemayehu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/102238/ > ----------------------------------------------------------- > > (Updated Aug. 12, 2011, 3:45 a.m.) > > > Review request for kdelibs. > > > Summary > ------- > > The attached patch is an alternate approach to address the issue of crashes > that arise from terminating an active thread than the one proposed at > https://git.reviewboard.kde.org/r/102179/. With this patch the function > "QHostInfo::lookupHost(QString, int)" avoids the use of QThread::terminate > with the following fairly simple changes: > > - Connect its finished signal to its parent deleteLater slot in the ctor so > that the thread is automatically deleted later. > - Store the looked up DNS info in the global cache to avoid unnecessary > queries for the same request. > - Check for cached DNS information and avoid doing reverse look ups before > resorting to performing DNS queries in a separate thread. > > > Diffs > ----- > > kio/kio/hostinfo.cpp 344b1d8 > > Diff: http://git.reviewboard.kde.org/r/102238/diff > > > Testing > ------- > > Local unit testing code. Tested both failing and working DNS lookup scenarios. > > > Thanks, > > Dawit > >