Ложная запись об изменении настройки "Минимальная частота двигателя 2" в лог

[san]

При анализе #181 в логе​ обнаружили странную запись:
May 27 14:22:56 smarthdcd[891]: config.cpp:174: Настройка Минимальная частота двигателя 2' изменена: 15.00 --> 8.00
После эксперимента в конфиге​ по прежнему Частота двигателя 2 = 15 Гц.

Предполагается, что запись в лог происходит ошибочно.

[andrei]

Конфиги и лог c версией smartHDC 717

[san]

Файлы Андрея из другого эксперимента, воспроизводящего баг.

[alx]

Думаю, что я нашел причину.

Дело в том, что любая настройка - это такой объект, который кроме своего значения (в даном случае минимальной частоты) хранит также вспомогательные метаданные - имя настройки, значение по умолчанию, минимум/максимум и т.п. А еще у нас есть параметры производительности, которые для единообразия реализованы как те же настройки.

В коде контроллера есть много мест, где параметр производительности устанавливается в значение минимума или максимума диапазона. Для этого в коде просто выполняется присваивание. Но, так как это объекты одного типа, компилятор копирует не только значение параметра, но и все метаданные тоже, включая имя настройки! Поэтому когда затем значение выбранной частоты меняется, вместо 'freqDirve2' мы видим в логе имя параметра, который ранее был присвоен выбранной частоте.

[alx]

In 719/smartHDC:

Неизменяемые метаданные настроек (имя настройки, дефолтное значение, минимум/максимум)
объявлены const. Настройкам добавлен оператор присваивания, копирующий только значение
настройки, но не метаданные. Closes #182.

[alx]

Предлагаю проверить r719 для уверенности, что баг пропал.

[andrei]

Обязательно!

[andrei]

Проверили, изменение минимальной частоты в логе перестало присутствовать.