Github user twogee commented on the pull request: https://github.com/apache/ant/commit/5aa1e8eff81ad6005e62587cdaaa962fce1105ea#commitcomment-27929054 In src/tests/junit/org/apache/tools/ant/taskdefs/RmicAdvancedTest.java: In src/tests/junit/org/apache/tools/ant/taskdefs/RmicAdvancedTest.java on line 476: I meant "instead of `try { ...; fail ... } catch { ... }`"; you can still put expected exception rule inside an `if {} else {}` or use assume to get rid of that if/else.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org For additional commands, e-mail: dev-h...@ant.apache.org