On 06/05/2016 14:18, Dmitry Samersoff wrote:
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("-"))"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.
-Alan