sys-file-reader: Tolerate nominal case size of 0 without warning.

The software at https://github.com/WizardMac/ReadStat seems to always write
0, so there's not much value in warning.

diff --git a/doc/dev/system-file-format.texi b/doc/dev/system-file-format.texi
index fb131f5274f0d61e90f9eefdf0c994c413b7e468..e52b9571166a25b477a8f6263f34d0960b9f0423 100644 (file)
--- a/doc/dev/system-file-format.texi
+++ b/doc/dev/system-file-format.texi
@@ -232,8 +232,8 @@ file's integer endianness (@pxref{System File Format}).
 Number of data elements per case.  This is the number of variables,
 except that long string variables add extra data elements (one for every
 8 characters after the first 8).  However, string variables do not
-contribute to this value beyond the first 255 bytes.   Further, system
-files written by some systems set this value to -1.  In general, it is
+contribute to this value beyond the first 255 bytes.   Further, some
+software always writes -1 or 0 in this field.  In general, it is
 unsafe for systems reading system files to rely upon this value.
 
 @item int32 compression;

diff --git a/src/data/sys-file-reader.c b/src/data/sys-file-reader.c
index 54788deb6e217bb918d8873077e2cb305ffbea77..5e5dc8b144f33c81d2b275be66504e62d833b762 100644 (file)
--- a/src/data/sys-file-reader.c
+++ b/src/data/sys-file-reader.c
@@ -861,7 +861,7 @@ sfm_decode (struct any_reader *r_, const char *encoding,
      amount that the header claims.  SPSS version 13 gets this
      wrong when very long strings are involved, so don't warn in
      that case. */
-  if (r->header.nominal_case_size != -1
+  if (r->header.nominal_case_size > 0
       && r->header.nominal_case_size != r->n_vars
       && r->info.version_major != 13)
     sys_warn (r, -1, _("File header claims %d variable positions but "