Hi Mandy,
thank you for your review.
although jdk.test.lib.compiler.CompilerUtils is easy to use, it introduces
dependency on jdk.compiler module and currently FragmentMetaspaceSimple.java
depends only on java.base module and I would prefer it remain so.
Thanks,
-- Igor
> On Aug 11, 2017,
On 8/10/17 9:02 PM, Igor Ignatyev wrote:
http://cr.openjdk.java.net/~iignatyev//8186095/webrev.00/index.html
hotspot/test/runtime/Metaspace/FragmentMetaspaceSimple.java
26 * @library /test/lib classes
27 * @build test.Empty
28 * @run driver ClassFileInstaller test.Empty
Hi Igor,
In (some of) the TEST.ROOT files, maintenance of the files can be
simplified by removing the version number from
the comment. The version that is in jaxp is reasonable: "# Minimum
jtreg version" and should be applied
to the hotspot, jdk, langtools, and nashorn TEST.ROOT files.
Thanks Ioi, and Igor, for clarifying.
David
On 11/08/2017 3:23 PM, Ioi Lam wrote:
On 8/10/17 9:46 PM, David Holmes wrote:
On 11/08/2017 2:31 PM, Igor Ignatyev wrote:
On Aug 10, 2017, at 9:22 PM, David Holmes
wrote:
Hi Igor,
On 11/08/2017 2:02 PM, Igor
On 8/10/17 9:46 PM, David Holmes wrote:
On 11/08/2017 2:31 PM, Igor Ignatyev wrote:
On Aug 10, 2017, at 9:22 PM, David Holmes
wrote:
Hi Igor,
On 11/08/2017 2:02 PM, Igor Ignatyev wrote:
http://cr.openjdk.java.net/~iignatyev//8186095/webrev.00/index.html
14
> On Aug 10, 2017, at 9:46 PM, David Holmes wrote:
>
> On 11/08/2017 2:31 PM, Igor Ignatyev wrote:
>>> On Aug 10, 2017, at 9:22 PM, David Holmes wrote:
>>>
>>> Hi Igor,
>>>
>>> On 11/08/2017 2:02 PM, Igor Ignatyev wrote:
On 11/08/2017 2:31 PM, Igor Ignatyev wrote:
On Aug 10, 2017, at 9:22 PM, David Holmes wrote:
Hi Igor,
On 11/08/2017 2:02 PM, Igor Ignatyev wrote:
http://cr.openjdk.java.net/~iignatyev//8186095/webrev.00/index.html
14 lines changed: 1 ins; 0 del; 13 mod;
Hi all,
> On Aug 10, 2017, at 9:22 PM, David Holmes wrote:
>
> Hi Igor,
>
> On 11/08/2017 2:02 PM, Igor Ignatyev wrote:
>> http://cr.openjdk.java.net/~iignatyev//8186095/webrev.00/index.html
>>> 14 lines changed: 1 ins; 0 del; 13 mod;
>> Hi all,
>> could you please review this
Hi Igor,
On 11/08/2017 2:02 PM, Igor Ignatyev wrote:
http://cr.openjdk.java.net/~iignatyev//8186095/webrev.00/index.html
14 lines changed: 1 ins; 0 del; 13 mod;
Hi all,
could you please review this small patch which bumps up jtreg version?
besides updating jib profiles and all TEST.ROOT
http://cr.openjdk.java.net/~iignatyev//8186095/webrev.00/index.html
> 14 lines changed: 1 ins; 0 del; 13 mod;
Hi all,
could you please review this small patch which bumps up jtreg version?
besides updating jib profiles and all TEST.ROOT files, the fix updates
10 matches
Mail list logo