Review Request 120926: Remove redundant code from KRun::KRunPrivate::isPromptNeeded()

[Arjun AK]

-----------------------------------------------------------
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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20141223/615c9b0c/attachment.html>