diff options
| author | 251 <13120787+251Labs@users.noreply.github.com> | 2018-01-02 01:06:23 +0100 |
|---|---|---|
| committer | 251 <13120787+251Labs@users.noreply.github.com> | 2018-01-02 01:06:23 +0100 |
| commit | 6dda059bde0798f54626108beab29d9d3834769c (patch) | |
| tree | 05892e56aef46f9cd812086527e3d6133de4bdc1 /src/qt | |
| parent | 63a4dc10876bfc61c2e87d35dcf17da2f0f8c316 (diff) | |
| download | bitcoin-6dda059bde0798f54626108beab29d9d3834769c.tar.xz | |
[qt] Simplifies boolean expression model && model->haveWatchOnly()
The boolean expression model && model->haveWatchOnly() can be simplified to model->haveWatchOnly(), because if (!model || !model->getOptionsModel()) { return; } guards against a potential dereferenced null pointer.
Diffstat (limited to 'src/qt')
| -rw-r--r-- | src/qt/transactionview.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/qt/transactionview.cpp b/src/qt/transactionview.cpp index b7e39e05ec..ab5b381953 100644 --- a/src/qt/transactionview.cpp +++ b/src/qt/transactionview.cpp @@ -365,7 +365,7 @@ void TransactionView::exportClicked() // name, column, role writer.setModel(transactionProxyModel); writer.addColumn(tr("Confirmed"), 0, TransactionTableModel::ConfirmedRole); - if (model && model->haveWatchOnly()) + if (model->haveWatchOnly()) writer.addColumn(tr("Watch-only"), TransactionTableModel::Watchonly); writer.addColumn(tr("Date"), 0, TransactionTableModel::DateRole); writer.addColumn(tr("Type"), TransactionTableModel::Type, Qt::EditRole); |