Filter: update filterData directly without copying

In FilterWidget2::updateFilter() a new FilterData object is generated
and then copied onto the filterData member variable. Instead, modify
filterData directly. This seems also more logical from a semantic
point of view: Do we want to reset fields that were not set by the
user?

Contains trivial whitespace fix.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>

1 files changed, 19 insertions, 22 deletions

diff --git a/desktop-widgets/filterwidget2.cpp b/desktop-widgets/filterwidget2.cpp
index fdf2bb843..4b4e722fe 100644
--- a/desktop-widgets/filterwidget2.cpp
+++ b/desktop-widgets/filterwidget2.cpp
@@ -75,29 +75,26 @@ FilterWidget2::FilterWidget2(QWidget* parent)
 
 void FilterWidget2::updateFilter()
 {
-	FilterData data;
+	filterData.validFilter = true;
+	filterData.minVisibility = ui->minVisibility->currentStars();
+	filterData.maxVisibility = ui->maxVisibility->currentStars();
+	filterData.minRating = ui->minRating->currentStars();
+	filterData.maxRating = ui->maxRating->currentStars();
+	filterData.minWaterTemp = ui->minWaterTemp->value();
+	filterData.maxWaterTemp = ui->maxWaterTemp->value();
+	filterData.minAirTemp = ui->minAirTemp->value();
+	filterData.maxWaterTemp = ui->maxWaterTemp->value();
+	filterData.from = ui->from->dateTime();
+	filterData.to = ui->to->dateTime();
+	filterData.tags = ui->tags->text().split(",", QString::SkipEmptyParts);
+	filterData.people = ui->people->text().split(",", QString::SkipEmptyParts);
+	filterData.location = ui->location->text().split(",", QString::SkipEmptyParts);
+	filterData.equipment = ui->equipment->text().split(",", QString::SkipEmptyParts);
+	filterData.invertFilter = ui->invertFilter->isChecked();
+	filterData.logged = ui->logged->isChecked();
+	filterData.planned = ui->planned->isChecked();
 
-	data.validFilter = true;
-	data.minVisibility = ui->minVisibility->currentStars();
-	data.maxVisibility = ui->maxVisibility->currentStars();
-	data.minRating = ui->minRating->currentStars();
-	data.maxRating = ui->maxRating->currentStars();
-	data.minWaterTemp = ui->minWaterTemp->value();
-	data.maxWaterTemp = ui->maxWaterTemp->value();
-	data.minAirTemp = ui->minAirTemp->value();
-	data.maxWaterTemp = ui->maxWaterTemp->value();
-	data.from = ui->from->dateTime();
-	data.to = ui->to->dateTime();
-	data.tags = ui->tags->text().split(",", QString::SkipEmptyParts);
-	data.people = ui->people->text().split(",", QString::SkipEmptyParts);
-	data.location = ui->location->text().split(",", QString::SkipEmptyParts);
-	data.equipment = ui->equipment->text().split(",", QString::SkipEmptyParts);
-	data.invertFilter = ui->invertFilter->isChecked();
-	data.logged = ui->logged->isChecked();
-	data.planned = ui->planned->isChecked();
-
-	filterData = data;
-	filterDataChanged(data);
+	filterDataChanged(filterData);
 }
 
 void FilterWidget2::updateLogged(int value) {