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

Arjun AK arjunak234 at gmail.com
Fri Oct 31 17:56:22 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120926/
-----------------------------------------------------------

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.cpp c623b58 
  src/widgets/krun_p.h 61660c0 

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/20141031/bd963d9d/attachment.html>


More information about the Kde-frameworks-devel mailing list