[patch #6363] rework settings into single struct

[Ben Pfaff]

Update of patch #6363 (project pspp):

                  Status:   Ready For Test/Review => Works For Me           

    _______________________________________________________

Follow-up Comment #1:

This looks OK to me.  A few nits:

In init_style, testing whether &styles[type] is NULL makes no sense: it will
always be false (unless styles is NULL and type is 0, but that shouldn't
happen).

Could each line in the_settings have a comment over on the right giving the
name of the corresponding member?  This will make it easer to read the list
and to update it as settings are added, changed, or removed.


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?6363>

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