[Octave-bug-tracker] [bug #52090] default values for missing GUI settings out of sync between settings dialog and actual behavior

[Mike Miller]

Follow-up Comment #3, bug #52090 (project octave):

I think the most important issue to address first is just to make sure that
the defaults in settings-dialog.cc agree with those used elsewhere. For
example


$ grep -Hn 'value.*connection.*toBool' libgui/src/*.cc
libgui/src/main-window.cc:193:        = settings->value
("news/allow_web_connection", true).toBool ();
libgui/src/main-window.cc:518:       ? settings->value
("news/allow_web_connection", true).toBool ()
libgui/src/settings-dialog.cc:511:  ui->checkbox_allow_web_connect->setChecked
(settings->value ("news/allow_web_connection", false).toBool ());


These should all be false.

Whether that means a refactoring of where these values or functions to wrap
how they are retrieved with their default values, up to you if you want to
look at that.

Secondarily, if you think we can minimize or eliminate entirely the
default-qt-settings file, that's a bonus.

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?52090>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/