Re: JDK 9 RFR of 8058679: More bad characters in BigIntegerTest

2014-09-19 Thread Brian Burkhalter
Hi Robert, What happened was that for whatever reason my version of the ‘patch’ command did not like your patch. I tried various things to get ‘patch’ to accept it but without success. Note that this had nothing to do with the modular restructuring of the source tree as the path is easily unshu

Re: JDK 9 RFR of 8058679: More bad characters in BigIntegerTest

2014-09-18 Thread Robert Gibson
Sorry about that, let's blame my crappy webmail ;) (Strange that the patch appears fine in the mail archives and the webrev?) Robert On Thursday, 18 September 2014, 3:18, Alan Bateman wrote: On 17/09/2014 21:48, Brian Burkhalter wrote: > Hello bad characters, > > Issue:https://bugs.openjdk

Re: JDK 9 RFR of 8058679: More bad characters in BigIntegerTest

2014-09-17 Thread Brian Burkhalter
On Sep 17, 2014, at 1:52 PM, Alan Bateman wrote: > On 17/09/2014 21:48, Brian Burkhalter wrote: >> Hello bad characters, >> >> Issue: https://bugs.openjdk.java.net/browse/JDK-8058679 >> Webrev: http://cr.openjdk.java.net/~bpb/8058679/webrev.00/ >> >> Tested with "-encoding US-ASCII”

Re: JDK 9 RFR of 8058679: More bad characters in BigIntegerTest

2014-09-17 Thread Alan Bateman
On 17/09/2014 21:48, Brian Burkhalter wrote: Hello bad characters, Issue: https://bugs.openjdk.java.net/browse/JDK-8058679 Webrev: http://cr.openjdk.java.net/~bpb/8058679/webrev.00/ Tested with "-encoding US-ASCII” passed to javac. \ufffd\ufffd Looks okay and if it compiles with -encoding US-

JDK 9 RFR of 8058679: More bad characters in BigIntegerTest

2014-09-17 Thread Brian Burkhalter
Hello bad characters, Issue: https://bugs.openjdk.java.net/browse/JDK-8058679 Webrev: http://cr.openjdk.java.net/~bpb/8058679/webrev.00/ Tested with "-encoding US-ASCII” passed to javac. Thanks, Brian