[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[patch #6358] Text import patch pre-review take 4
From: |
Ben Pfaff |
Subject: |
[patch #6358] Text import patch pre-review take 4 |
Date: |
Fri, 07 Mar 2008 06:06:34 +0000 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.1) Gecko/20061205 Iceweasel/2.0.0.1 (Debian-2.0.0.1+dfsg-1) |
Follow-up Comment #18, patch #6358 (project pspp):
Jason Stover writes:
> psppire: src/data/variable.c:179: var_set_name: Assertion
`var_is_plausible_name (name, 0)' failed.
Very clever: you picked a data file that used the PSPP reserved word "and" as
a variable name. The function for generating unique names wasn't smart enough
to notice this, but the dictionary refused to allow adding the variable, hence
the assertion failure. Good catch!
I checked in a change that fixes the problem. After that, I was able to
import your data file without trouble, although the actual import operation
could certainly be a little faster.
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/patch/?6358>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/
- [patch #6358] Text import patch pre-review take 4, Ben Pfaff, 2008/03/06
- [patch #6358] Text import patch pre-review take 4, John Darrington, 2008/03/06
- [patch #6358] Text import patch pre-review take 4, Ben Pfaff, 2008/03/06
- [patch #6358] Text import patch pre-review take 4, Jason H Stover, 2008/03/06
- [patch #6358] Text import patch pre-review take 4, Ben Pfaff, 2008/03/07
- [patch #6358] Text import patch pre-review take 4,
Ben Pfaff <=
- [patch #6358] Text import patch pre-review take 4, John Darrington, 2008/03/07
- [patch #6358] Text import patch pre-review take 4, Ben Pfaff, 2008/03/08
- [patch #6358] Text import, Ben Pfaff, 2008/03/16
- [patch #6358] Text import, John Darrington, 2008/03/16
- [patch #6358] Text import, Ben Pfaff, 2008/03/16
- [patch #6358] Text import, Ben Pfaff, 2008/03/16