some code style improvements

This commit is contained in:
Alberto Griggio 2017-03-05 18:10:42 +01:00
parent f8bf8d18dc
commit 43e32e81ef
4 changed files with 10 additions and 6 deletions

View File

@ -70,7 +70,7 @@ bool DynamicProfileRule::operator<(const DynamicProfileRule &other) const
}
bool DynamicProfileRule::matches(const rtengine::ImageMetaData *im)
bool DynamicProfileRule::matches(const rtengine::ImageMetaData *im) const
{
return (iso(im->getISOSpeed())
&& fnumber(im->getFNumber())

View File

@ -48,7 +48,7 @@ public:
};
DynamicProfileRule();
bool matches(const rtengine::ImageMetaData *im);
bool matches(const rtengine::ImageMetaData *im) const;
bool operator<(const DynamicProfileRule &other) const;
int serial_number;

View File

@ -517,16 +517,15 @@ void DynamicProfilePanel::on_button_edit()
void DynamicProfilePanel::save()
{
auto &rules = profileStore.getDynamicProfileRules();
rules.clear();
std::vector<DynamicProfileRule> rules;
int serial = 1;
for (auto row : treemodel_->children()) {
rules.emplace_back(to_rule(row, serial++));
}
if (!storeDynamicProfileRules(rules)) {
printf("Error in saving dynamic profile rules\n");
rules.clear();
} else {
profileStore.setDynamicProfileRules(rules);
printf("Saved %d dynamic profile rules\n", int(rules.size()));
}
}

View File

@ -208,11 +208,16 @@ public:
return internalDynamicEntry;
}
std::vector<DynamicProfileRule> &getDynamicProfileRules()
const std::vector<DynamicProfileRule> &getDynamicProfileRules() const
{
return dynamicRules;
}
void setDynamicProfileRules(const std::vector<DynamicProfileRule> &r)
{
dynamicRules = r;
}
void addListener(ProfileStoreListener *listener);
void removeListener(ProfileStoreListener *listener);