RFR(XXS) : 8179930: jdk.test.lib.artifacts.ArtifactResolver::resolve should return Map instead of HashMap

George Triantafillou george.triantafillou at oracle.com
Tue May 9 17:55:55 UTC 2017


Hi Igor,

Looks good.

-George

On 5/9/2017 1:20 PM, Igor Ignatyev wrote:
> http://cr.openjdk.java.net/~iignatyev//8179930/webrev.00/index.html
>> 8 lines changed: 1 ins; 0 del; 7 mod;
> Hi all,
>
> could you please review this small patch which changes jdk.test.lib.artifacts.ArtifactResolver::resolve signature to return a Map instead of HasMap and updates the tests accordingly?
> I have also changed the argument type from raw Class to Class<?>, so we don't need to have casts in the method.
>
> webrev: http://cr.openjdk.java.net/~iignatyev//8179930/webrev.00/index.html
> JBS: https://bugs.openjdk.java.net/browse/JDK-8179930
> testing: the affected tests (hotspot/test/applications)
>
> -- Igor



More information about the hotspot-dev mailing list