[Pspp-cvs] pspp/src/data ChangeLog sys-file-writer.c

[Ben Pfaff]

CVSROOT:        /cvsroot/pspp
Module name:    pspp
Changes by:     Ben Pfaff <blp> 07/11/19 01:06:06

Modified files:
        src/data       : ChangeLog sys-file-writer.c 

Log message:
        Correctly calculate n_missing_values field when writing variables that
        include a range of missing values.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/pspp/src/data/ChangeLog?cvsroot=pspp&r1=1.169&r2=1.170
http://cvs.savannah.gnu.org/viewcvs/pspp/src/data/sys-file-writer.c?cvsroot=pspp&r1=1.37&r2=1.38

Patches:
Index: ChangeLog
===================================================================
RCS file: /cvsroot/pspp/pspp/src/data/ChangeLog,v
retrieving revision 1.169
retrieving revision 1.170
diff -u -b -r1.169 -r1.170
--- ChangeLog   11 Nov 2007 05:51:40 -0000      1.169
+++ ChangeLog   19 Nov 2007 01:06:05 -0000      1.170
@@ -1,3 +1,13 @@
+2007-11-18  Ben Pfaff  <address@hidden>
+
+       Properly write variables that include a range of missing values to
+       system files.  Thanks to Guido Gay <address@hidden> for reporting
+       this bug.
+
+       * sys-file-writer.c (write_variable): Correctly calculate
+       n_missing_values field when writing variables that include a range
+       of missing values.
+
 2007-11-10  Ben Pfaff  <address@hidden>
 
        Cleanups and bug fixes devised while writing up documentation.

Index: sys-file-writer.c
===================================================================
RCS file: /cvsroot/pspp/pspp/src/data/sys-file-writer.c,v
retrieving revision 1.37
retrieving revision 1.38
diff -u -b -r1.37 -r1.38
--- sys-file-writer.c   11 Nov 2007 05:51:41 -0000      1.37
+++ sys-file-writer.c   19 Nov 2007 01:06:05 -0000      1.38
@@ -421,7 +421,7 @@
   /* Number of missing values.  If there is a range, then the
      range counts as 2 missing values and causes the number to be
      negated. */
-  write_int (w, mv_has_range (mv) ? 2 - mv_n_values (mv) : mv_n_values (mv));
+  write_int (w, mv_has_range (mv) ? -2 - mv_n_values (mv) : mv_n_values (mv));
 
   /* Print and write formats. */
   write_format (w, *var_get_print_format (v), seg0_width);