hg: jdk7/tl: 6934759: Add langtools testing to jprt control builds

2010-03-12 Thread kelly . ohair
Changeset: c60ed0f6d91a Author:ohair Date: 2010-03-12 17:44 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/rev/c60ed0f6d91a 6934759: Add langtools testing to jprt control builds Reviewed-by: jjg ! make/jprt.properties

Re: Sponsor for 6666666: A better implementation of Character.isSupplementaryCodePoint

2010-03-12 Thread Martin Buchholz
OK, next round of review. I changed my UTF-8 changes to be behavior-preserving, removing any hint of controversy, and renamed the patch to "utf8-twiddling". I got Ulf in my head, and can't stop micro-optimizing. I added a new micro-optimizing patch for Bits.java. Please file a bug. 6934268: Bett

hg: jdk7/tl: 6934712: run langtools jtreg tests from top level test/Makefile

2010-03-12 Thread jonathan . gibbons
Changeset: bbd817429100 Author:jjg Date: 2010-03-12 15:22 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/rev/bbd817429100 6934712: run langtools jtreg tests from top level test/Makefile Reviewed-by: ohair ! test/Makefile

Re: Sponsor for 6666666: A better implementation of Character.isSupplementaryCodePoint

2010-03-12 Thread Martin Buchholz
On Fri, Mar 12, 2010 at 05:04, Ulf Zibis wrote: > Am 12.03.2010 02:46, schrieb Martin Buchholz: > public static boolean isISOControl(int codePoint) { > // Optimized form of: > // (codePoint >= 0x && codePoint <= 0x001F) || > // (codePoint >= 0x007F && codePo

Re: Sponsor for 6666666: A better implementation of Character.isSupplementaryCodePoint

2010-03-12 Thread Martin Buchholz
On Thu, Mar 11, 2010 at 13:14, Ulf Zibis wrote: > Am 11.03.2010 20:38, schrieb Martin Buchholz: > - Maybe better:  "... using a single {...@code char}". Done. > - Why don't you like using the new isBMPCodePoint() for > isSupplementaryCodePoint() and toUpperCaseCharArray() ? I now use it for the

hg: jdk7/tl/jdk: 2 new changesets

2010-03-12 Thread kelly . ohair
Changeset: bf6eb240e718 Author:ohair Date: 2010-03-12 09:03 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/bf6eb240e718 6933294: Fix some test/Makefile issues around Linux ARCH settings, better defaults Reviewed-by: jjg ! test/Makefile ! test/ProblemList.txt Changeset: cda

hg: jdk7/tl/langtools: 6934224: update langtools/test/Makefile

2010-03-12 Thread jonathan . gibbons
Changeset: f856c0942c06 Author:jjg Date: 2010-03-12 12:00 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/langtools/rev/f856c0942c06 6934224: update langtools/test/Makefile Reviewed-by: ohair ! make/jprt.properties ! test/Makefile

Re: TEST: java/nio/channels/AsynchronousSocketChannel/Basic.java

2010-03-12 Thread Kelly O'Hair
Filed bug 6934585 -kto On Mar 12, 2010, at 12:06 AM, Alan Bateman wrote: Kelly O'Hair wrote: I'm having problems with this test on Solaris 10 X86 and Fedora 9 32bit. Ring any bells? -kto I haven't seen this failure but looking at the test now, the connect can fail immediately which woul

Re: Sponsor for 6666666: A better implementation of Character.isSupplementaryCodePoint

2010-03-12 Thread Ulf Zibis
Hi Martin, is that 666 a fake bug id? I still can't see it in the public bugparade: * This bug is not available. More information is available at -http://developers.sun.com/resources/bugsFAQ.html#s4q2 -Ulf Am 11.03.2010 20:38, schrieb Martin Buchholz: Ulf, your changes woul

Re: Sponsor for 6666666: A better implementation of Character.isSupplementaryCodePoint

2010-03-12 Thread Ulf Zibis
Am 12.03.2010 02:46, schrieb Martin Buchholz: http://mercurial.selenic.com/wiki/MqExtension http://hgbook.red-bean.com/read/managing-change-with-mercurial-queues.html Ah, I see mq is part of hg. Another thing to learn, but sounds good. Unfortunately there seems no GUI for it. I run Tortois

Re: TEST: java/nio/channels/AsynchronousSocketChannel/Basic.java

2010-03-12 Thread Alan Bateman
Kelly O'Hair wrote: I'm having problems with this test on Solaris 10 X86 and Fedora 9 32bit. Ring any bells? -kto I haven't seen this failure but looking at the test now, the connect can fail immediately which would cause both of the test failures - can you create a bug and I'll fix this. -A