first impression

[Tom Tromey]

>>>>> "Thomas" == Thomas Reilly <treilly@livesoftware.com> writes:

Thomas> - java.io.PrintStream.println(String) doesn't check to see if
Thomas> String is null and use "null" instead (like PrintWriter does).
Thomas> This causes a null pointer exception when it calls getBytes()
Thomas> on the string.

Thanks.  I'll check in a fix for this.

Thomas> - Inner classes don't work.  What's entailed in fixing that
Thomas> and is their anyone working on it?

I don't know what exactly is involved.  I believe we're planning to do
this, but I don't know when.  I don't think anyone is working on it
right now.

Thomas> How do you guys want to handle bug fixes like the PrintStream
Thomas> thing?  Its only a one-liner but should I send in a patch
Thomas> anyway?

Yes.  Please see contribute.html for information.  Basically: we want
a patch with the correct indentation style, a description of what the
patch does and why it is required, and a ChangeLog entry.  The web
page says how to use "diff" to make the patch (we want a specific diff
format).

Thomas> The install target of the Makefile only installs the zip
Thomas> program.  To get everything to work I had to put the
Thomas> libgcj.zip in my CLASSPATH and install libgcgjc.a and libgcj.a
Thomas> in the lib directory of my egcs installation.  That would
Thomas> probably make a good FAQ item somewhere.

`make install' should have installed these files in the correct place.
And for my test builds, it does.
Can you find out why it didn't work for you?

Tom