Rename rewriting failure to fail

seanjensengrey / kiama

Automatically exported from code.google.com/p/kiama

GNU Lesser General Public License v3.0

0 stars 0 forks source link

Rename rewriting failure to fail #40

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

We originally used failure because Stratego's fail clashed with JUnit's fail.  
But failure clashes with 
the Scala parser library's failure, which is used more in this kind of code. 
So, since we don't really 
use JUint's fail and we are moving away from direct use of  JUnit hopefully, 
switch back to fail.

Original issue reported on code.google.com by inkytonik on 17 Nov 2009 at 4:52

GoogleCodeExporter commented 9 years ago

Original comment by inkytonik on 19 Nov 2009 at 2:24

Changed state: Started
Added labels: Milestone-Release1.0
Removed labels: Milestone-Release1.1

GoogleCodeExporter commented 9 years ago

Done (r624f7811ab).  We still need to use fail in tests (it's ScalaTests's 
fail, not JUnit's now) but there is no point 
in giving up fail (which is what is used in Stratego) to make the tests a bit 
easier to write.

Original comment by inkytonik on 23 Nov 2009 at 3:14

Changed state: Resolved