Index: sources/storage/Settings.h =================================================================== diff -u -ra82ee4f326b26c4369f1306f867edc1bca6dabbe -ra04fd119778f0483cba5139a11e3c6bad7f2324a --- sources/storage/Settings.h (.../Settings.h) (revision a82ee4f326b26c4369f1306f867edc1bca6dabbe) +++ sources/storage/Settings.h (.../Settings.h) (revision a04fd119778f0483cba5139a11e3c6bad7f2324a) @@ -30,8 +30,12 @@ namespace Storage { // TODO - IMPORTANT: this MVC needs to change to configurations in oppose to Settings. -// It named settings because it suppose to be like QSettings, -// but since we have settings it is mixing up with that and is very confusing. +// It named settings because it suppose to be like QSettings, +// but since we have settings it is mixing up with that and is very confusing. +// FIXME- IMPORTANT: This set of Settings (MVC) design is not correct: +// - the data structure should change in Model from column based (keys, values) to tree/map based +// - the singleton should be the controller not the Model, to protect the model usage, and avoid the shared memory issues. +// - the model class should be private class, or an object defined in this class, not a global class. class Settings {