JDK 10 RFR of JDK-8183378: Refactor java/lang/System/MacEncoding/MacJNUEncoding.sh to java

Felix Yang felix.yang at oracle.com
Thu Jul 6 02:16:00 UTC 2017


Hi Amy,

     looks fine. Just one comment on sentence below. "LOCALE" looks to 
be a local variable, though used several times. Switch to usual naming?

   50         final String LOCALE = args[2];

-Felix
On 2017/7/6 9:47, Amy Lu wrote:
> java/lang/System/MacEncoding/MacJNUEncoding.sh
>
> Please review this patch to refactor the shell test to java.
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8183378
> webrev: http://cr.openjdk.java.net/~amlu/8183378/webrev.00/
>
> Thanks,
> Amy



More information about the core-libs-dev mailing list