RFR(S): 8192927: os::dir_is_empty is incorrect on Windows

Ioi Lam ioi.lam at oracle.com
Thu Jan 4 02:02:00 UTC 2018


Hi Calvin,

The code changes look good. I would suggesting running the tests in our 
test harness several times so you can run on different windows hosts.


For the test case, I think there are too many use of "DoesntMatter" 
which may be confusing. I would suggest adding

     String classList[] = {"java/lang/Object"};

and replacingall 'TestCommon.list("DoesntMatter")' with classList.

Also,

    File subDir = new File(longDir, "DoesntMatter");
->
    File subDir = new File(longDir, "subdir");


Thanks
- Ioi



On 12/20/17 2:00 PM, Calvin Cheung wrote:
> I've updated the change slightly:
> - replaced the do-while loop with a while loop;
> - free(search_path) should be os::free(search_path)
>
> updated webrev: http://cr.openjdk.java.net/~ccheung/8192927/webrev.01/
>
> thanks,
> Calvin
>
> On 12/19/17, 5:01 PM, Calvin Cheung wrote:
>> bug: https://bugs.openjdk.java.net/browse/JDK-8192927
>>
>> This change is targeted for JDK 11.
>> Please refer to the comment in the bug for a description of the 
>> change. It also handles path longer than MAX_PATH.
>>
>> webrev: http://cr.openjdk.java.net/~ccheung/8192927/webrev.00/
>>
>> Testing:
>>     hs-tier1, hs-tier2 on linux-x64, windows-x64, macosx
>>     appcds tests on the above platforms + sparcv9.
>>
>> thanks,
>> Calvin



More information about the hotspot-runtime-dev mailing list