-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120926/
-----------------------------------------------------------
(Updated Dec. 23, 2014, 5:56 p.m.)
Status
------
This change has been marked as submitted.
Review request for KDE Frameworks and David Faure.
Repository: kio
Description
-------
Since `setShowScriptExecutionPrompt()` is used to specify whether a prompt
should be shown or not, we don't neither need to check it again nor do we need
to call `setRunExecutable()` inside KIO. The caller is anyway forced to do
something like:
if (value == "alwaysAsk") {
run->setShowScriptExecutionPrompt(true);
} else {
run->setRunExecutables(value == "execute");
}
Diffs
-----
src/widgets/krun.h cd5bcaa
Diff: https://git.reviewboard.kde.org/r/120926/diff/
Testing
-------
Thanks,
Arjun AK
_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
