From dfffbe7ef353c8920f10f197bf2ac8a00fea3cf7 Mon Sep 17 00:00:00 2001
From: parrt <parrt@antlr.org>
Date: Fri, 14 May 2010 15:07:50 -0800
Subject: [PATCH] improved funcs; args back in; push/pop to func def.

[git-p4: depot-paths = "//depot/code/antlr4/main/": change = 6868]
---
 .../v4/tool/templates/codegen/Java/Java.stg    | 18 ++++--------------
 1 file changed, 4 insertions(+), 14 deletions(-)

diff --git a/tool/resources/org/antlr/v4/tool/templates/codegen/Java/Java.stg b/tool/resources/org/antlr/v4/tool/templates/codegen/Java/Java.stg
index 34c289349..120dcdd23 100644
--- a/tool/resources/org/antlr/v4/tool/templates/codegen/Java/Java.stg
+++ b/tool/resources/org/antlr/v4/tool/templates/codegen/Java/Java.stg
@@ -37,25 +37,17 @@ public class <p.name> {
 RuleFunction(f,code,decls,context) ::= <<
 <context>
 
-<if(f.isStartRule)>
-<if(f.modifiers)><f.modifiers:{f | <f> }><else>public final <endif><if(f.retType)><f.retType><else>void<endif> <f.name>(<f.args; separator=", ">) {
+<if(f.modifiers)><f.modifiers:{f | <f> }><else>public final <endif><if(f.retType)><f.retType><else>void<endif> <f.name>(<f.args:{a|<a>, }>BitSet _follow) {
+<!	<if(f.scope)><f.scope.name>.push(new <f.scope.name>);<endif> !>
 <if(f.retType)>
     _ctx.push(new <f.context.name>(<f.args:{a|<a.name>}; separator=",">));
-    _<f.name>(null);
-    return _ctx.pop();
-<else>
-    _<f.name>(null);
 <endif>
-}
-
-<endif>
-<if(f.modifiers)><f.modifiers:{f | <f> }><else>public final <endif><if(f.retType)><f.retType><else>void<endif> <if(f.isStartRule)>_<endif><f.name>(BitSet _follow) {
-<!	<if(f.scope)><f.scope.name>.push(new <f.scope.name>);<endif> !>
     <decls; separator="\n">
 	try {
 	    <code>
 	}
 	finally {
+        <if(f.retType)>return _ctx.pop();<endif>
 	}
 }
 >>
@@ -148,9 +140,7 @@ cases(ttypes) ::= <<
 >>
 
 InvokeRule(r) ::= <<
-<if(r.ctxName)>_ctx.push(new <r.ctxName>(<r.argExprs; separator=",">));<endif>
-<if(r.label)><r.label> = <endif><r.name>(<r.follow.name>);
-<if(r.ctxName)>_ctx.pop();<endif>
+<if(r.label)><r.label> = <endif><r.name>(<r.argExprs:{a|<a>, }><r.follow.name>);
 >>
 
 MatchToken(m) ::= <<