RFR: CODETOOLS-7902215: Follow-up generification and misc, part 3

[dmitry.bessonov at oracle.com]

On 12/07/2018 23:28, Jonathan Gibbons wrote:
> 
> 
> 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.

Updated the patch; new webrev is at
http://cr.openjdk.java.net/~dbessono/7902215/01/webrev/

Besides src/com/sun/javatest/util/Properties.java also used {@code} in 
src/com/sun/javatest/TestEnvironment.java


dmitry

> 
> -- Jon