On 06/05/2016 14:18, Dmitry Samersoff wrote:
Everybody,

Please, see updated webrev.

http://cr.openjdk.java.net/~dsamersoff/JDK-8155091/webrev.05/

Removed ToolProvider interface. run(String ... args) function removed or
renamed to runWithArgs.

Fixed two more tests, missed on previous iteration.

Updated webrev looks good. Minor nit in checkForUnsupportedOptions is that a space are crept into a few of the `if` statements, e.g. "if (! s.matches("[0-9]+"))" and "if (! s.startsWith("-"))"

-Alan

Reply via email to