- User Since
- Mar 5 2015, 12:44 PM (206 w, 6 d)
Tue, Feb 19
Looks good, just some minor improvement suggestions for the unittest.
If some code is deleting this job from a slot connected to it, that code needs to be fixed. This hack isn't a fix, it will only create more problems..
Mon, Feb 18
Bleh, phabricator sucks. It does show a list of commits, but no way to view (and review) each commit's diff separately?
No objection from me, but I'm no utf-8 expert.
Sun, Feb 17
Ah I see, we copy to .part only when the destination actually exists. I had missed that in my earlier comment. Makes sense, actually.
kio_thumbnail is a separate process (like all kioslaves) so a crash doesn't bring down the user-visible application.
Sat, Feb 16
Thanks for working on this, here's my review.
Fri, Feb 15
Tue, Feb 12
Mon, Feb 11
Your reasoning seems to forget that there is a first confirmation, in case of overwriting, in the form of the "Overwrite/Skip/Cancel" dialog?
Sun, Feb 10
Sat, Feb 9
Wed, Feb 6
This should only be done if KRun::setEnableExternalBrowser(false) wasn't called, otherwise it will break KParts' BrowserRun subclass and Konqueror's KonqRun (who use KRun to actually find out the mimetype of the file, not to launch another browser).
his means adding a bool d->m_externalBrowserEnabled in KRun, since right now we can't know that it was called if the settings don't actually define an external browser.
This makes no sense to me. The whole point of " based on the contents of the URL " is to open images in an image viewer, PDFs in a PDF viewer, etc.
So hardcoding text/html breaks this and opens everything in a webbrowser -- if that's what you want, set a browser app, not "based on the contents of the URL".
Tue, Feb 5
Mon, Feb 4
https://build.kde.org/job/Frameworks/view/Everything/job/kconfig/job/kf5-qt5%20FreeBSDQt5.12/ green !!! :-)
Well, blue, because blue is the new green (!?).
Sun, Feb 3
Using exec() from a job implementation is a big no no, it creates an unexpected re-entrancy, source of all sorts of problems. This cannot be shipped as is.
Recreate socket as suggested
remove another pair of add/remove methods
also add lower-level unittest
Sat, Feb 2
Add m_startMode to AkThread to make quitThread() a no-op if NoThread.
Right, this would fail if the part was not using QFile, indeed. Still, at the level of the KParts frameworks itself it's all fine ;)
A better name would be "hasOnlySpaces" (with the opposite return value), but OK.
Sorry for the delay