whiteknight/io_cleanup1 branch ready for testing

[James E Keenan]

On 8/15/12 7:52 PM, James E Keenan wrote:

>
> Well, I now have a caveat concerning the io_cleanup1 branch. I have just
> noticed that in the output of 'make' we have been getting Perl
> "uninitialized value" warnings since at least commit 1ed2945 tested by
> me on July 22. The warnings look like this:
>
> /usr/local/bin/perl tools/build/c2str.pl src/pmc/stringiterator.c >
> src/pmc/stringiterator.str
> /usr/local/bin/perl tools/build/c2str.pl src/pmc/structview.c >
> src/pmc/structview.str
> Use of uninitialized value $str in hash element at tools/build/c2str.pl
> line 155, <$IN> line 733.
> Use of uninitialized value $str in concatenation (.) or string at
> tools/build/c2str.pl line 156, <$IN> line 733.
> Use of uninitialized value $str in concatenation (.) or string at
> tools/build/c2str.pl line 163, <$IN> line 733.
> cc -I./include -I./include/pmc [long string of options] -o
> src/ops/core_ops.o -c src/ops/core_ops.c
>
> I will try to look into this.
>

The warnings appear to be being generated when tools/build/c2str.pl 
processes src/pmc/stringhandle.c, whose source file is 
src/pmc/stringhandle.pmc.  The problematic commit appears to be this:

commit 6c44a3a9a8296ca34b60fafaf2d313b83ab6fddd
Author: Whiteknight <wknight8111 at gmail.com>
Date:   Fri Jul 20 20:59:41 2012 -0400

     Change to the readline logic, in pursuit of a permanent fix. A few 
other small changes