Fixes #3483. define_method with empty body throws RuntimeError in interpreter

enebo · enebo · commit 1ee900744952 · 2015-11-23T16:48:13.000-06:00
diff --git a/core/src/main/java/org/jruby/ast/IterNode.java b/core/src/main/java/org/jruby/ast/IterNode.java
@@ -66,7 +66,7 @@ public IterNode(ISourcePosition position, ArgsNode args, Node body, StaticScope
         super(position, args != null && args.containsVariableAssignment || body != null && body.containsVariableAssignment);
 
         this.varNode = args;
-        this.bodyNode = body;
+        this.bodyNode = body == null ? NilImplicitNode.NIL : body;
         this.scope = scope;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ public IterNode(ISourcePosition position, ArgsNode args, Node body, StaticScope`
`66`	`66`	`super(position, args != null && args.containsVariableAssignment \|\| body != null && body.containsVariableAssignment);`
`67`	`67`
`68`	`68`	`this.varNode = args;`
`69`		`- this.bodyNode = body;`
	`69`	`+ this.bodyNode = body == null ? NilImplicitNode.NIL : body;`
`70`	`70`	`this.scope = scope;`
`71`	`71`	`}`
`72`	`72`