[GitHub] ant pull request #:

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] ant pull request #:

bodewig
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: [hidden email]
For additional commands, e-mail: [hidden email]