diff options
| author | Noel Grandin <noel@peralex.com> | 2016-03-14 13:41:38 +0200 |
|---|---|---|
| committer | Noel Grandin <noel@peralex.com> | 2016-03-15 08:27:25 +0200 |
| commit | b47cb646ff2a62fcd3fac0e453a7261bbaefbcb7 (patch) | |
| tree | 49a2dc78e998baa77212776d7d94fb8d72dc0f00 /extensions/source | |
| parent | 89e0663c55f7f1763536a345d63111115c71ef26 (diff) | |
loplugin:constantparam
Change-Id: I270e068b3c83e966e741b0a072fecce9d92d53f5
Diffstat (limited to 'extensions/source')
| -rw-r--r-- | extensions/source/dbpilots/listcombowizard.cxx | 11 | ||||
| -rw-r--r-- | extensions/source/dbpilots/listcombowizard.hxx | 2 |
2 files changed, 6 insertions, 7 deletions
diff --git a/extensions/source/dbpilots/listcombowizard.cxx b/extensions/source/dbpilots/listcombowizard.cxx index 2a7e529f8097..4c87f062c0a3 100644 --- a/extensions/source/dbpilots/listcombowizard.cxx +++ b/extensions/source/dbpilots/listcombowizard.cxx @@ -214,18 +214,17 @@ namespace dbp return true; } - Reference< XNameAccess > OLCPage::getTables(bool _bNeedIt) + Reference< XNameAccess > OLCPage::getTables() { Reference< XConnection > xConn = getFormConnection(); - DBG_ASSERT(!_bNeedIt || xConn.is(), "OLCPage::getTables: should have an active connection when reaching this page!"); - (void)_bNeedIt; + DBG_ASSERT(xConn.is(), "OLCPage::getTables: should have an active connection when reaching this page!"); Reference< XTablesSupplier > xSuppTables(xConn, UNO_QUERY); Reference< XNameAccess > xTables; if (xSuppTables.is()) xTables = xSuppTables->getTables(); - DBG_ASSERT(!_bNeedIt || xTables.is() || !xConn.is(), "OLCPage::getTables: got no tables from the connection!"); + DBG_ASSERT(xTables.is() || !xConn.is(), "OLCPage::getTables: got no tables from the connection!"); return xTables; } @@ -233,7 +232,7 @@ namespace dbp Sequence< OUString > OLCPage::getTableFields() { - Reference< XNameAccess > xTables = getTables(true); + Reference< XNameAccess > xTables = getTables(); Sequence< OUString > aColumnNames; if (xTables.is()) { @@ -320,7 +319,7 @@ namespace dbp m_pSelectTable->Clear(); try { - Reference< XNameAccess > xTables = getTables(true); + Reference< XNameAccess > xTables = getTables(); Sequence< OUString > aTableNames; if (xTables.is()) aTableNames = xTables->getElementNames(); diff --git a/extensions/source/dbpilots/listcombowizard.hxx b/extensions/source/dbpilots/listcombowizard.hxx index 07d859660074..cd7c8a82b941 100644 --- a/extensions/source/dbpilots/listcombowizard.hxx +++ b/extensions/source/dbpilots/listcombowizard.hxx @@ -89,7 +89,7 @@ namespace dbp bool isListBox() { return static_cast<OListComboWizard*>(getDialog())->isListBox(); } protected: - css::uno::Reference< css::container::XNameAccess > getTables(bool _bNeedIt); + css::uno::Reference< css::container::XNameAccess > getTables(); css::uno::Sequence< OUString > getTableFields(); }; |