rooty/LibrePilot
1
0
Fork 0
mirror of https://bitbucket.org/librepilot/librepilot.git synced 2025-01-18 03:52:11 +01:00
Code Issues Releases Activity

CameraStab UI: some reformatting and comments

Browse Source
This commit is contained in:
Oleg Semyonov 2012-08-21 22:09:53 +03:00
parent 84d55ca416
commit ffd5220071
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
ground/openpilotgcs/src/plugins/config/configcamerastabilizationwidget.cpp
View File

@ -151,11 +151,13 @@ void ConfigCameraStabilizationWidget::refreshWidgetsValues(UAVObject *obj)
*/
void ConfigCameraStabilizationWidget::updateObjectsFromWidgets()
{
// Save state of the module enable checkbox first
HwSettings *hwSettings = HwSettings::GetInstance(getObjectManager());
quint8 value = m_camerastabilization->enableCameraStabilization->isChecked() ?
// Save state of the module enable checkbox first.
// Do not use setData() member on whole object, if possible, since it triggers
// unnessesary UAVObect update.
quint8 enableModule = m_camerastabilization->enableCameraStabilization->isChecked() ?
HwSettings::OPTIONALMODULES_ENABLED : HwSettings::OPTIONALMODULES_DISABLED;
hwSettings->setOptionalModules(HwSettings::OPTIONALMODULES_CAMERASTAB,value);
HwSettings *hwSettings = HwSettings::GetInstance(getObjectManager());
hwSettings->setOptionalModules(HwSettings::OPTIONALMODULES_CAMERASTAB, enableModule);
// Update mixer channels which were mapped to camera outputs in case they are
// not used for other function yet
@ -218,6 +220,8 @@ void ConfigCameraStabilizationWidget::updateObjectsFromWidgets()
}
} while(widgetUpdated);
// FIXME: Should not use setData() to prevent double updates.
// It should be refactored after the reformatting of MixerSettings UAVObject.
mixerSettings->setData(mixerSettingsData);
ConfigTaskWidget::updateObjectsFromWidgets();