---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111776/#review37114
---
This review has been submitted with commit
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111776/#review36982
---
This review has been submitted with commit
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111776/
---
(Updated Aug. 2, 2013, 12:37 p.m.)
Status
--
This change has been
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111776/#review36984
---
Sorry for my slow review which led you to commit meanwhile,
On Aug. 2, 2013, 12:53 p.m., David Faure wrote:
kio/kio/clipboardupdater.cpp, line 108
http://git.reviewboard.kde.org/r/111776/diff/2/?file=175012#file175012line108
The name sounds like it will clear the clipboard.
removeUrlsFromClipboard would be better
Well this
On Aug. 2, 2013, 12:53 p.m., David Faure wrote:
kio/kio/clipboardupdater.cpp, line 108
http://git.reviewboard.kde.org/r/111776/diff/2/?file=175012#file175012line108
The name sounds like it will clear the clipboard.
removeUrlsFromClipboard would be better
Dawit
On Aug. 2, 2013, 12:53 p.m., David Faure wrote:
kio/kio/copyjob.cpp, line 2190
http://git.reviewboard.kde.org/r/111776/diff/2/?file=175014#file175014line2190
Where's this instance deleted?
Should there be a deleteLater() in all code paths of slotResult()?
Dawit
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111776/#review36990
---
This review has been submitted with commit
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111776/
---
Review request for kdelibs and David Faure.
Description
---
This