----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126865/#review91631 -----------------------------------------------------------
Ship it! Yeah... I guess what happened under regular use was that the first request by the first ioslave started and then failed to connect to the cache cleaner. The cache cleaner then became ready and started to get used after a few requests or so. - Andreas Hartmetz On Jan. 23, 2016, 11 p.m., David Faure wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126865/ > ----------------------------------------------------------- > > (Updated Jan. 23, 2016, 11 p.m.) > > > Review request for KDE Frameworks and Andreas Hartmetz. > > > Repository: kio > > > Description > ------- > > The old code was doing connectToServer+waitForConnected, but this > fails immediately if connectToServer sets an error like > QLocalSocket::ConnectionRefusedError. > waitForConnected() is only meaningful if we are in ConnectingState. > > We do have to try multiple times (no way to wait until precisely > the time when it start listening), but not using waitForConnected, > so I used msleep(100) in the loop instead. > > I could have used DBus activation of course (to block until it's ready), > but I assume DBus is not wanted here since that's not what's used for > communication in the first place. > > Change-Id: Ib9ee4699a44daa3016a72a24c6e816b940e1ed5c > > > Diffs > ----- > > src/ioslaves/http/http.cpp 76da711cbb3eee9ba801a6e8117d78aaee467626 > > Diff: https://git.reviewboard.kde.org/r/126865/diff/ > > > Testing > ------- > > kill kio_http_cache_cleaner; run unittest using KIO::get; watch kio_http > output. > > Before: > HTTPProtocol::sendCacheCleanerCommand: Could not connect to cache cleaner, > not updating stats of this cache file. > > After: > HTTPProtocol::sendCacheCleanerCommand: Successfully connected to cache > cleaner. > > > Thanks, > > David Faure > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
