Re: Review Request : Java exe doesn't process args ending Back slash

[Kumar Srinivasan]

Hi Jayashree,

I have assigned a JBS issue to this: JDK-8006536, for your reference.
I have attached a modified patch to that issue.

There are other issues as well that needs to be resolved with \ parsing.
We will address this once we are done with our JDK8 Milestone feature
commitments.

Thanks
Kumar

On 16-01-2013 6:26 AM, Kumar Srinivasan wrote:

Hello Jayashree,

I see the issue, I have been busy with other things, I will revisit your
patch in the coming week.

Kumar

On 20-12-2012 12:07 AM, Kumar Srinivasan wrote:

Hello Jayashree,

a. you are referencing a bug which has already been fixed, is there a
    new one for this ?

b. with regards to the fix, I don't quite understand the issue, could you please

    provide a  use case ?

c. your regression test does not seem to be accurate it behaves the same with or without your fix, also you will need to provide a C++ test case in cmdtoargs.c

    as well see the bottom of that file.


Thanks
Kumar

Hi All,

Java.exe doesn't seems to process arguments ending with back slashes
well , in windows only .

I have added test scenario and changeset in the below webrev .

http://cr.openjdk.java.net/~jviswana/7188114/webrev.01/

This seems to be introduced after the bug fix for 7188114 has be made
into jdk8 .

Thanks and Regards,
Jayashree Viswanathan

Hi Kumar ,

a. I am referencing an old bug because , that bug fix has caused this regression .

b. The use case is when there are backslashes at the end args for a java command using say -Dtest.argEndingInBackslash=a\\\\

JavaVM args:
    version 0x00010002, ignoreUnrecognized is JNI_FALSE, nOptions is 5
    option[ 0] = '-Dsun.java.launcher.diag=true'
    option[ 1] = '-Djava.class.path=.'
    option[ 2] = '-Dtest.argEndingInBackslash=a'
    option[ 3] = '-Dsun.java.command=TestCmdLineParsing'
    option[ 4] = '-Dsun.java.launcher=SUN_STANDARD'
74439 micro seconds to InitializeJVM
Main class is 'TestCmdLineParsing'
App's argc is 0
9182 micro seconds to load main class
----_JAVA_LAUNCHER_DEBUG----
value of test.argEndingInBackslash = a

c. Sorry , I seem to have missed something , the above test case should help you exhibit the problem . Can you please let me know where to find or add such C++ test cases , as In the test cases bucket I know off is jtreg or JCKs only at the moment .

Thanks and Regards,
Jayashree Viswanathan

Thanks for the response !

Regards,
Jayashree Viswanathan