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