pspp-commits
[Top][All Lists]
Advanced

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

[Pspp-commits] [SCM] GNU PSPP branch, master, updated. v0.6.1-1865-gc581


From: Ben Pfaff
Subject: [Pspp-commits] [SCM] GNU PSPP branch, master, updated. v0.6.1-1865-gc58173e
Date: Fri, 04 Feb 2011 04:41:47 +0000

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU PSPP".

The branch, master has been updated
       via  c58173ea116eed59ce96a4b233f2d2d8ceb50b26 (commit)
       via  6c091c8123879cb0cf385e167e468eb82ac69d2c (commit)
       via  32bcf82d7525c3e50d63064ff32b1545989836b8 (commit)
       via  a628323f67aa963f1e0ec866dc6cd9ede022de82 (commit)
       via  0c335df036f26ba66ad3d6d4fd0fef6b7f8bf359 (commit)
       via  acc4324d6da262e63d4dd811cce24ba64cff1fc7 (commit)
       via  207dcaa849d7c1d5090f198e9c86f9bfc484a89e (commit)
       via  f994fab43f6aeb00cfb40d24c4c4c6ed518a05b9 (commit)
       via  871dc2a4ffe7bd1c1b0a7c5a451ce9e23d4b7969 (commit)
      from  95b6394d56fe347634a207f2e65a3ef9cfe05ca8 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit c58173ea116eed59ce96a4b233f2d2d8ceb50b26
Author: Ben Pfaff <address@hidden>
Date:   Thu Feb 3 20:41:40 2011 -0800

    sys-file-reader: Add tests.

commit 6c091c8123879cb0cf385e167e468eb82ac69d2c
Author: Ben Pfaff <address@hidden>
Date:   Sun Jan 30 15:41:12 2011 -0800

    sys-file-reader: Elevate invalid number of labels from warning to error.
    
    There's no way to figure out when the labels end when the count is wrong,
    so this has to be an error.

commit 32bcf82d7525c3e50d63064ff32b1545989836b8
Author: Ben Pfaff <address@hidden>
Date:   Sun Jan 30 15:27:06 2011 -0800

    sys-file-reader: Avoid redundant "within system file" in messages.
    
    All system file message already state the system file's name, which should
    be enough identification.

commit a628323f67aa963f1e0ec866dc6cd9ede022de82
Author: Ben Pfaff <address@hidden>
Date:   Sun Jan 30 15:42:51 2011 -0800

    sys-file-reader: Improve debugging usefulness of error messages.
    
    I found that these changes made it easier to debug problems with system
    files while I was developing tests.

commit 0c335df036f26ba66ad3d6d4fd0fef6b7f8bf359
Author: Ben Pfaff <address@hidden>
Date:   Sun Jan 30 15:40:17 2011 -0800

    sys-file-reader: Really enable warnings for invalid variable formats.
    
    With these operator in the wrong order, I never saw any warnings, although
    other people might have seen different behavior.

commit acc4324d6da262e63d4dd811cce24ba64cff1fc7
Author: Ben Pfaff <address@hidden>
Date:   Sun Jan 30 15:29:38 2011 -0800

    sys-file-reader: Add missing periods at end of error messages.
    
    Most of the error messages already ended in periods, so this adds them to
    the others for consistency.

commit 207dcaa849d7c1d5090f198e9c86f9bfc484a89e
Author: Ben Pfaff <address@hidden>
Date:   Sun Jan 30 15:35:48 2011 -0800

    sys-file-reader: Check that multiple response set names begin with `$'.
    
    This invariant was being checked by an assertion in dict_add_mrset(), so
    this change avoids an assertion failure for invalid system files.

commit f994fab43f6aeb00cfb40d24c4c4c6ed518a05b9
Author: Ben Pfaff <address@hidden>
Date:   Sun Jan 30 15:33:30 2011 -0800

    sys-file-reader: Fix double-free on read of partial record.
    
    partial_record() does a longjmp() to code near the top of
    sys_file_casereader_read() that unrefs the case, but this code at the
    end of sys_file_casereader_read() would have already unref'ed it once in
    that case, resulting in a double-free.

commit 871dc2a4ffe7bd1c1b0a7c5a451ce9e23d4b7969
Author: Ben Pfaff <address@hidden>
Date:   Sun Jan 30 20:34:32 2011 -0800

    hash-functions: Use C instead of preprocessor to test size of double.
    
    I don't see a reason that this has to be done as an #if directive.  The
    compiler should be easily able to optimize out the branch that will never
    be taken.  This allows us to get rid of a configure check, so it seems
    worthwhile to me.

-----------------------------------------------------------------------

Summary of changes:
 Smake                           |    1 +
 configure.ac                    |    4 +-
 doc/dev/system-file-format.texi |    4 +-
 src/data/sys-file-reader.c      |   66 +-
 src/libpspp/hash-functions.c    |   27 +-
 tests/atlocal.in                |    1 +
 tests/automake.mk               |    7 +
 tests/data/sack.c               |  499 ++++++
 tests/data/sys-file-reader.at   | 3315 +++++++++++++++++++++++++++++++++++++++
 9 files changed, 3874 insertions(+), 50 deletions(-)
 create mode 100644 tests/data/sack.c
 create mode 100644 tests/data/sys-file-reader.at


hooks/post-receive
-- 
GNU PSPP



reply via email to

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