RFR: CODETOOLS-7902215: Follow-up generification and misc, part 3
Jonathan Gibbons
jonathan.gibbons at oracle.com
Thu Jul 12 22:28:33 UTC 2018
On 07/12/2018 07:27 AM, dmitry.bessonov at oracle.com wrote:
> Please review the following follow-up fix related to generics usage in
> JTHarness code; also compatibility with JTReg codebase has been restored.
>
> JBS: https://bugs.openjdk.java.net/browse/CODETOOLS-7902215
> Webrev: http://cr.openjdk.java.net/~dbessono/7902215/webrev/
>
>
> dmitry
Standard comments: there's more cleanup that can be done, like replacing
Iterator.
Watch for long lines: they make webrevs harder to read.
The edit to src/com/sun/javatest/util/Properties.java is somewhat
regressive,
and should be fixed. It is preferable to use '{@code Map<String, String>}'
instead of the raw HTML equivalent.
-- Jon
More information about the jtharness-dev
mailing list