diff options
author | Roland Knall <rknall@gmail.com> | 2019-11-14 23:04:59 +0100 |
---|---|---|
committer | Roland Knall <rknall@gmail.com> | 2019-11-14 22:54:29 +0000 |
commit | 5f8ed1126a812b49ec5f6728739892c21d0fb3ec (patch) | |
tree | 1e92773cd69ab50ad332cae37c433f84d28c90e8 /ui/qt/models/coloring_rules_model.cpp | |
parent | 725200d6507efadd853b3c21a01c84982ef8a00e (diff) |
Qt: Fix various LGTM.com reported issues
Mostly shadowed variables or similar minor issues
Change-Id: I98a76eb4cac1ba1356a239a0ed7beb550ced0651
Reviewed-on: https://code.wireshark.org/review/35091
Petri-Dish: Roland Knall <rknall@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Roland Knall <rknall@gmail.com>
Diffstat (limited to 'ui/qt/models/coloring_rules_model.cpp')
-rw-r--r-- | ui/qt/models/coloring_rules_model.cpp | 14 |
1 files changed, 12 insertions, 2 deletions
diff --git a/ui/qt/models/coloring_rules_model.cpp b/ui/qt/models/coloring_rules_model.cpp index 7d574a2d26..6dbe77943f 100644 --- a/ui/qt/models/coloring_rules_model.cpp +++ b/ui/qt/models/coloring_rules_model.cpp @@ -60,6 +60,16 @@ ColoringRuleItem::ColoringRuleItem(const ColoringRuleItem& item) { } +ColoringRuleItem& ColoringRuleItem::operator=(ColoringRuleItem& rhs) +{ + disabled_ = rhs.disabled_; + name_ = rhs.name_; + filter_ = rhs.filter_; + foreground_ = rhs.foreground_; + background_ = rhs.background_; + return *this; +} + // Callback for color_filters_clone. void color_filter_add_cb(color_filter_t *colorf, gpointer user_data) @@ -454,9 +464,9 @@ bool ColoringRulesModel::dropMimeData(const QMimeData *data, Qt::DropAction acti QJsonArray dataArray = encodedData.object()["coloringrules"].toArray(); - for ( int row = 0; row < dataArray.count(); row++ ) + for ( int datarow = 0; datarow < dataArray.count(); datarow++ ) { - QJsonObject entry = dataArray.at(row).toObject(); + QJsonObject entry = dataArray.at(datarow).toObject(); if ( ! entry.contains("foreground") || ! entry.contains("background") || ! entry.contains("filter") ) continue; |