Applied with the following fixes:
On Fri, 2009-04-24 at 21:29 +0200, Tomek Grabiec wrote:
> diff --git a/include/jit/expression.h b/include/jit/expression.h
> index c131766..ff9866f 100644
> --- a/include/jit/expression.h
> +++ b/include/jit/expression.h
> @@ -273,6 +273,7 @@ struct expression *anewarray_expr(struct object *, struct
> expression *);
> struct expression *multianewarray_expr(struct object *);
> struct expression *arraylength_expr(struct expression *);
> struct expression *instanceof_expr(struct expression *, struct object *);
> +struct expression *exception_ref_expr();
Oops. This should have been in the previous patch. I fixed that up.
> @@ -20,17 +21,7 @@
>
> static bool is_exception_handler(struct basic_block *bb)
> {
> - struct methodblock *method = bb->b_parent->method;
> - int i;
> -
> - for (i = 0; i < method->exception_table_size; i++) {
> - struct exception_table_entry *eh = &method->exception_table[i];
> -
> - if (eh->handler_pc == bb->start)
> - return true;
> - }
> -
> - return false;
> + return (exception_find_entry(bb->b_parent->method, bb->start) != NULL);
Unnecessary parenthesis around the expression. Fixed.
> class ExceptionsTest extends TestCase {
> public static void main(String args[]) {
> - testCatchCompilation();
> + int result;
> +
> + result = testCatchCompilation();
> + assertEquals(result, 2);
This is wrong. You should never do assertEquals() from main(). Fixed.
------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensign option that enables unlimited
royalty-free distribution of the report engine for externally facing
server and web deployment.
http://p.sf.net/sfu/businessobjects
_______________________________________________
Jatovm-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/jatovm-devel
