From c69ca4df80c9c74aa842b7f1fb3c44b22ae3232e Mon Sep 17 00:00:00 2001 From: Dirk Hohndel Date: Wed, 20 Mar 2019 07:31:24 -0700 Subject: Core: simplify ConnectionListModel The complicated setup with the AddressRole is unnecessary. All we want to be able to do is get the index of a specific text in the list. In hindsight I am puzzled why I implemented this in such a complex fashion. Signed-off-by: Dirk Hohndel --- core/downloadfromdcthread.cpp | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) (limited to 'core/downloadfromdcthread.cpp') diff --git a/core/downloadfromdcthread.cpp b/core/downloadfromdcthread.cpp index 214854342..e2db8b66e 100644 --- a/core/downloadfromdcthread.cpp +++ b/core/downloadfromdcthread.cpp @@ -287,12 +287,8 @@ QStringList DCDeviceData::getProductListFromVendor(const QString &vendor) int DCDeviceData::getMatchingAddress(const QString &vendor, const QString &product) { - for (int i = 0; i < connectionListModel.rowCount(); i++) { - QString address = connectionListModel.address(i); - if (address.contains(product)) - return i; - } - return -1; + Q_UNUSED(vendor) + return connectionListModel.indexOf(product); } DCDeviceData *DownloadThread::data() -- cgit v1.2.3-70-g09d2