BUG: Fix applySettings() to emit "settingChanged()" only if needed

commontk / CTK

A set of common support code for medical imaging, surgical navigation, and related purposes.

https://commontk.org

Apache License 2.0

855 stars 490 forks source link

BUG: Fix applySettings() to emit "settingChanged()" only if needed #1177

Closed jcfr closed 8 months ago

jcfr commented 8 months ago

This issue was discovered while working on fixing ctkSettingsPanelTest1 failure.

Further improve the fix integrated in c8ce0dec ("BUG: Fix incorrect changed settings indicator", 2018-07-04) and update ctkSettingsPanelTest1 accordingly.

cc: @jamesobutler