test_string_limits_ro & nil vectors

[Sam S. <sd...@gn...>]

Bruno,
what should be placed into arg->string?
the doc says that it's "data vector, a not-reallocated simple-string".
nil vector's data vector is NIL, so NIL is put there.
the original string is thereby lost - but it must be returned by
WRITE-STRING, WRITE-LINE, NSTRING-UPCASE, NSTRING-DOWNCASE and
NSTRING-CAPITALIZE.

Moreover, test_string_limits_rw() appears to assume that arg->string is
indeed the original nil string and does not crash only because an error
is already signaled earlier.

I propose to make stringarg contain the string instead of the useless
NIL.

Agreed?

-- 
Sam Steingold (http://www.podval.org/~sds) running w2k
<http://www.camera.org> <http://www.iris.org.il> <http://www.memri.org/>
<http://www.mideasttruth.com/> <http://www.honestreporting.com>
What was the best thing before sliced bread?