pspp-dev
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[patch #5583] NPAR TESTS


From: Ben Pfaff
Subject: [patch #5583] NPAR TESTS
Date: Sat, 25 Nov 2006 01:36:39 +0000
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1) Gecko/20061024 Iceweasel/2.0 (Debian-2.0+dfsg-1)

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

Have you checked out the algorithms described on spss.com against what SPSS
Statistical Algorithms says?  The latter might be dated; who knows.

In value_dup, I'd prefer to calculate the size as MIN(width, sizeof *val).

In compare_var_index, please use less-than and greater-than to figure out the
comparison result; if we ever change the `index' member to an unsigned type
(which would make a lot of sense), we'll really appreciate not having to hunt
down this bug.

I didn't look very closely at the new files you introduced.  I'll assume that
they do the right thing.

    _______________________________________________________

Reply to this item at:

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

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





reply via email to

[Prev in Thread] Current Thread [Next in Thread]