Merge #12074: [qt] Optimizes boolean expression model && model->haveWatchOnly()
commita1136f0cb449122e5591e29a7f71c1ec7dc0937b
authorJonas Schnelli <dev@jonasschnelli.ch>
Thu, 4 Jan 2018 07:58:37 +0000 (3 21:58 -1000)
committerJonas Schnelli <dev@jonasschnelli.ch>
Thu, 4 Jan 2018 07:58:46 +0000 (3 21:58 -1000)
tree2432fefa541d3e217d51f76072ea5f68a5241f31
parentc991b304dee368f506cfee27ddaa333f1f82c518
parent6dda059bde0798f54626108beab29d9d3834769c
Merge #12074: [qt] Optimizes boolean expression model && model->haveWatchOnly()

6dda059bd [qt] Simplifies boolean expression model && model->haveWatchOnly() (251)

Pull request description:

  This PR optimizes the boolean expression `model && model->haveWatchOnly()` to `model->haveWatchOnly()`.

  The boolean expression can be optimized because the method `TransactionView::exportClicked` already guards against a potential dereferenced null pointer by returning early if `model` is null.
  https://github.com/bitcoin/bitcoin/blob/63a4dc10876bfc61c2e87d35dcf17da2f0f8c316/src/qt/transactionview.cpp#L351-L353

Tree-SHA512: 8bdd0d05bf879745fa39d3ca7524471720ae08ceee9427d5a08776e7b56d18542ae87a6991cd6779e232305f504fdfc77223702b72ecbe231f5f5e98453456dd
src/qt/transactionview.cpp