RFR 8124977 cmdline encoding challenges on Windows
vlashch at microsoft.com
Wed Nov 25 22:38:00 UTC 2015
Please find updated webreview here:
From: Kumar Srinivasan [mailto:kumar.x.srinivasan at oracle.com]
Sent: Sunday, November 22, 2015 8:14 AM
To: Martin Sawicki <marcins at microsoft.com>
Cc: Kirk Shoop <Kirk.Shoop at microsoft.com>; Vladimir Shcherbakov <vlashch at microsoft.com>; core-libs-dev Libs <core-libs-dev at openjdk.java.net>
Subject: Re: RFR 8124977 cmdline encoding challenges on Windows
Hi Martin, et. al.,
Sorry for not getting back earlier, I am very busy right now with my other large commitments for JDK9.
I will sponsor this "enhancement/bug fix" sometime in the new year, meanwhile, there is the changeset  which is likely to cause merge conflicts, and perhaps logic issues.
> Hi all
> Here's an updated webrev attempting to take into account the various pieces of feedback we have received:
> (Vladimir Shcherbakov is now working on this from our side)
> Looking forward to any other feedback.
> -----Original Message-----
> From: core-libs-dev [mailto:core-libs-dev-bounces at openjdk.java.net] On
> Behalf Of Kumar Srinivasan
> Sent: Thursday, June 25, 2015 6:26 AM
> To: Kirk Shoop (MS OPEN TECH) <Kirk.Shoop at microsoft.com>
> Cc: Valery Kopylov (Akvelon) <v-valkop at microsoft.com>; core-libs-dev
> Libs <core-libs-dev at openjdk.java.net>
> Subject: Re: RFR 8124977 cmdline encoding challenges on Windows
> Hi Kirk,
> Thanks for proposing this change.
> If you notice all the posix calls are wrapped in JLI_* this gives us the ability to use "W" functions. I almost got it done, several years ago, but we upgraded to VS2010 and my work based on VS2003 keeled over, meanwhile my focus was "shifted" to something else.
> main.c: is really envisioned to be a stub compiled by the tool
> launchers, like java, javac, javah, jar etc. I prefer to see all the
> heavy logic in this file moved to the platform specific file
> For the reason specified above we need to move fprintf or any naked posix calls to JLI_* indirections.
> I don't see any tests ? The tests must be written in java and placed in jdk/test/tools/launcher, there is a helper framework TestHelper.java.
> There are other changes in nio, charsets etc, this will be reviewed by my colleague specializing in that area (Sherman) cc'ed.
> On 6/22/2015 2:01 PM, Kirk Shoop (MS OPEN TECH) wrote:
>> This webrev intends to address interaction between Windows console and java apps.
>> Two switches were added that change the behavior of the launcher. The defaults do not change the launcher behavior.
>> -Dwindows.UnicodeConsole=true - switches on Unicode support in the Windows console. This optional switch causes the launcher to call GetCommandLineW() and parse the arguments in unicode. It also modifies how the codepage for console output is selected.
>> -Dfile.encoding.unicode="UTF-8" - identifies Unicode charset to use; If not specified, UTF-8 is used by default. Ignored when windows.UnicodeConsole is not set to true. When the first switch is used, this optional switch allows the codepage for console output to be controlled.
>> I would like to get feedback on the approach here and any additional work that is required solve these particular Unicode issues on Windows.
More information about the core-libs-dev