[patch #6109] Improvements to sys file reader, writer

[John Darrington]

Follow-up Comment #1, patch #6109 (project pspp):

I've not actually tried any of these patches.  But here are my comments:

In por-writer-vls.patch, perhaps "255" should be  replaced by something more
meaningfull (maybe MIN_VERY_LONG_STRING - 1)

Regarding the assignment of the display width in sysfile writer, I agree the
current code is bizarre.
I think that a string variable's display width when it is created should be
set to be equal to the format width unless that width exceeds some maximum. 
This should be done in variable.c ; I don't see any need for sysfile-writer
to tamper with this parameter.


    _______________________________________________________

Reply to this item at:

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

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