On 16/02/2018 14:39, Michal Vala wrote:
:
That sounds reasonable. Here's the webrev:
http://cr.openjdk.java.net/~shade/8170120/webrev.01/
It fixes JImageVerifyTest so I've removed it from ProblemList.
note: JImageExtractTest still failing on not-empty-dir(JDK-8170114)
and JImageListTest fails on simplest list test. I'll take a look at it
later.
The changes look good, I just wonder if it would be better to specify
{0} as file rather than file.getName(). Just thinking about a script
running this tool failing because the file path is wrong, I think it
could be useful to have it in error.
-Alan