Skip to content

Commit eaac352

Browse files
committed
data_container: avoid variable shadowing
1 parent dd9697a commit eaac352

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

ulog_cpp/data_container.cpp

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -43,17 +43,17 @@ void DataContainer::headerComplete()
4343

4444
// try to resolve all fields for params
4545
for (auto& it : _default_parameters) {
46-
auto& parameter = it.second;
47-
parameter.field().resolveDefinition(_message_formats, 0);
46+
auto& parameter_value = it.second;
47+
parameter_value.field().resolveDefinition(_message_formats, 0);
4848
}
4949

5050
for (auto& it : _initial_parameters) {
51-
auto& parameter = it.second;
52-
parameter.field().resolveDefinition(_message_formats, 0);
51+
auto& parameter_value = it.second;
52+
parameter_value.field().resolveDefinition(_message_formats, 0);
5353
}
5454

55-
for (auto& parameter : _changed_parameters) {
56-
parameter.field().resolveDefinition(_message_formats, 0);
55+
for (auto& parameter_value : _changed_parameters) {
56+
parameter_value.field().resolveDefinition(_message_formats, 0);
5757
}
5858

5959
_header_complete = true;
@@ -100,10 +100,10 @@ void DataContainer::parameter(const Parameter& parameter_arg)
100100
return;
101101
}
102102
if (_header_complete) {
103-
auto parameter = parameter_arg;
103+
auto parameter_value = parameter_arg;
104104
// if header is complete, we can resolve definition here
105-
parameter.field().resolveDefinition(_message_formats, 0);
106-
_changed_parameters.push_back(parameter);
105+
parameter_value.field().resolveDefinition(_message_formats, 0);
106+
_changed_parameters.push_back(parameter_value);
107107
} else {
108108
_initial_parameters.insert({parameter_arg.field().name(), parameter_arg});
109109
}

0 commit comments

Comments
 (0)