On 02/19/2018 02:07 PM, Alan Bateman wrote:
On 19/02/2018 12:09, Michal Vala wrote:


Yes, true. Attached.
This looks good except the "does not exist" case which needs this change:

-                throw TASK_HELPER.newBadArgs("err.not.a.jimage", 
file.getName());
+                throw TASK_HELPER.newBadArgs("err.not.a.jimage", file);

If you agree then I will include it in your patch and push it.

yes please. thanks


--
Michal Vala
OpenJDK QE
Red Hat Czech

Reply via email to