Thanks for the comment, I made the updates and will verify with JPRT.

Valerie


On 8/4/2016 9:07 PM, Mandy Chung wrote:
On Aug 4, 2016, at 7:20 AM, Sean Mullan <sean.mul...@oracle.com> wrote:

On 08/04/2016 12:42 AM, Mandy Chung wrote:
A passing comment:

The test can simply specify @modules jdk.crypto.mscapi and this test will only 
run when jdk.crypto.mscapi is present; in other words, line 45-48 are no longer 
needed.
Does that also mean line 27 can be removed:

27  * @requires os.family == “windows"

I think it should also be removed but you should verify it to make sure jtreg 
selects this test on windows only (where jdk.crypto.mscap1 is present).

Mandy


Reply via email to