Thank you,

Misha


On 05/23/2018 04:13 PM, Igor Ignatyev wrote:
Hi Misha,

looks good to me.

-- Igor

On May 23, 2018, at 4:13 PM, mikhailo <mikhailo.seledt...@oracle.com> wrote:

Hi Calvin,

   Thank you for review. I will fix the issue with vmTestbase_nsk_stress (merge 
issue) and will remove the blank line at line 1273 prior to check in.


Misha


On 05/23/2018 03:15 PM, Calvin Cheung wrote:
Hi Misha,

I've compared the file.list from your closed webrev with the one from this open 
webrev and didn't see any missing files.
Also spot checked a few copyright headers and they look good.

Regarding TEST.groups, why was the following removed?
1160 vmTestbase_nsk_stress = \
1161   vmTestbase/nsk/stress

Could you also remove the extra blank line added at line 1273?

thanks,
Calvin

On 5/21/18, 11:34 AM, Mikhailo Seledtsov wrote:
Please review this change that will open source VM default method tests.
These tests have been used internally for a while, and are now being open 
sourced. Since this is not an creation of new tests, we would like to keep the 
changes during this review to a minimum required for open sourcing these tests, 
such as major issues and integration blockers. If you have other feedback 
regarding improvements to these tests, please file RFE(s) that will be 
addressed later in order of priority.

Here is what was done for this change:
   1. Moved the tests to OpenJDK repository to the specified directory location 
and structure.
   3. Updated Copyright statements accordingly.
   4. Updated "@library" statements accordingly.
   5. Updated TEST.groups and a HotSpot test make file

   JBS:    https://bugs.openjdk.java.net/browse/JDK-8199255
   Webrev: http://cr.openjdk.java.net/~mseledtsov/8199255.01/

   Testing:
       1. Ran the following tests on open-only repository and build, using "make 
run-test" (Linux-x64)
          vmTestbase_vm_defmeth
          All PASS

       2. Automated multip-platform test system (usual 4 platforms):
          - vmTestbase_vm_defmeth
          - hs-tier{1,2}
          In progress


Thank you,
Misha


Reply via email to