This is the mail archive of the java-patches@gcc.gnu.org mailing list for the Java project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: More java.net stuff ...


>>>>> "Michael" == Michael Koch <konqueror@gmx.de> writes:

Michael> Another java.net patch. Please review.

Thanks for doing this work.
Unfortunately right now I'm out of gcj t-shirts, or I'd send you one :-(.
I'll call the t-shirt store this week and see if they'll print a new
run for me (last time they were really reluctant, so I don't know).

There's a couple things in this patch.

Michael> +	* java/net/DatagramSocketImpl.jav
Michael> +	(getOption): Removed. Compiler bug seems to be fixed.
Michael> +	(setOption): Removed. Compiler bug seems to be fixed.

As I recall these methods were added to Classpath because gcj couldn't
correctly compile all of Classpath to bytecode without them.  I think
this bug is still in gcj.  Could you test to see?  The way to test it
is to remove these methods from Classpath, and then configure and
build Classpath so that it uses gcj to do the build.

In particular, you won't be able to see this bug given the way that
libgcj is built (one class at a time).

Michael> +   * Sets a ContenHandlerFactory

Typo.

Tom


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]