Hi Alan,
Thank you for the review. May I ask your help for sponsoring this fix?
Best Regards,
Toshio Nakamura
Alan Bateman wrote on 2020/04/24 17:50:39:
> On 24/04/2020 09:33, Toshio 5 Nakamura wrote:
> > Hi all,
> >
> > Please review this fix.
> > Also, I'd like to ask a sponsor of the fix, since I'm not a committer.
> >
> > Issue:
https://urldefense.proofpoint.com/v2/url?u=https-3A__bugs.openjdk.java.net_browse_JDK-2D8243453=DwICaQ=jf_iaSHvJObTbx-siA1ZOg=EVbFABcgo-X99_TGI2-qsMtyulHUruf8lAzMlVpVRqw=cT0541m2fenmr7m1ceVW_jsLw9PFcETHyrVZmBIBT7I=2MCap8HFtEpR-ewTpkte08Ae4EKczAH07RvscWGqU6g=
> > Webrev:
https://urldefense.proofpoint.com/v2/url?u=http-3A__cr.openjdk.java.net_-7Etnakamura_8243453_webrev.00_=DwICaQ=jf_iaSHvJObTbx-siA1ZOg=EVbFABcgo-X99_TGI2-qsMtyulHUruf8lAzMlVpVRqw=cT0541m2fenmr7m1ceVW_jsLw9PFcETHyrVZmBIBT7I=gzf38aY-kf9TZz9Ehd_8mqgr3KEkfVAdP9rVO0T8Fro=
> >
> > Under Windows Japanese or Chinese, encoding of command line option
> > cannot be assumed as UTF-8.
> > This fix changes from "(*env)->NewStringUTF()" to "NewPlatformString()"
> > to call Helper method.
> >
> This looks correct to me (and apologies for this bug, I don't know why I
> used NewStringUTF when adding this option in JDK 9).
>
> -Alan.
>