From 60cf71a56420ced8e61ab9759d20652eab0ad9e5 Mon Sep 17 00:00:00 2001
From: Sam Harwell <sam@tunnelvisionlabs.com>
Date: Mon, 26 Nov 2012 11:04:30 -0600
Subject: [PATCH] RuleActionFunction and RuleSempredFunction templates generate
 private methods

---
 .../org/antlr/v4/tool/templates/codegen/Java/Java.stg  | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 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 07a1c953b..b2c3239ea 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
@@ -183,8 +183,11 @@ public <p.name>(TokenStream input) {
 }
 >>
 
+/* This generates a private method since the actionIndex is generated, making an
+ * overriding implementation impossible to maintain.
+ */
 RuleActionFunction(r, actions) ::= <<
-public void <r.name>_action(<r.ctxType> _localctx, int actionIndex) {
+private void <r.name>_action(<r.ctxType> _localctx, int actionIndex) {
 	switch (actionIndex) {
 	<actions:{index|
 case <index>: <actions.(index)> break;}; separator="\n">
@@ -192,8 +195,11 @@ case <index>: <actions.(index)> break;}; separator="\n">
 }
 >>
 
+/* This generates a private method since the predIndex is generated, making an
+ * overriding implementation impossible to maintain.
+ */
 RuleSempredFunction(r, actions) ::= <<
-public boolean <r.name>_sempred(<r.ctxType> _localctx, int predIndex) {
+private boolean <r.name>_sempred(<r.ctxType> _localctx, int predIndex) {
 	switch (predIndex) {
 	<actions:{index|
 case <index>: return <actions.(index)>;}; separator="\n">