[Truffle] Fail hard on unknown AST nodes.

* Implements visitOpAsgnConstDeclNode as visitOpAsgnOrNode.
jruby · Nov 7, 2016 · ed3c459 · ed3c459 · bjfish · Nov 7, 2016
1 parent 7512988
commit ed3c459
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/truffle/src/main/java/org/jruby/truffle/parser/BodyTranslator.java b/truffle/src/main/java/org/jruby/truffle/parser/BodyTranslator.java
@@ -223,6 +223,7 @@
 import org.jruby.truffle.parser.ast.NilParseNode;
 import org.jruby.truffle.parser.ast.NthRefParseNode;
 import org.jruby.truffle.parser.ast.OpAsgnAndParseNode;
+import org.jruby.truffle.parser.ast.OpAsgnConstDeclParseNode;
 import org.jruby.truffle.parser.ast.OpAsgnOrParseNode;
 import org.jruby.truffle.parser.ast.OpAsgnParseNode;
 import org.jruby.truffle.parser.ast.OpElementAsgnParseNode;
@@ -2158,7 +2159,7 @@ public RubyNode visitMultipleAsgnNode(MultipleAsgnParseNode node) {
         RubyNode rhsTranslated;
 
         if (rhs == null) {
-            rhsTranslated = nilNode(source, sourceSection);
+            throw new UnsupportedOperationException("null rhs");
         } else {
             rhsTranslated = rhs.accept(this);
         }
@@ -2541,6 +2542,12 @@ public RubyNode visitOpAsgnAndNode(OpAsgnAndParseNode node) {
         return addNewlineIfNeeded(node, ret);
     }
 
+    @Override
+    public RubyNode visitOpAsgnConstDeclNode(OpAsgnConstDeclParseNode node) {
+        // TODO (eregon, 7 Nov. 2016): Is there any semantic difference?
+        return visitOpAsgnOrNode(new OpAsgnOrParseNode(node.getPosition(), node.getFirstNode(), node.getSecondNode()));
+    }
+
     @Override
     public RubyNode visitOpAsgnNode(OpAsgnParseNode node) {
         final ISourcePosition pos = node.getPosition();
@@ -2903,7 +2910,6 @@ public RubyNode visitRescueNode(RescueParseNode node) {
                         final RescueSplatNode rescueNode = new RescueSplatNode(context, fullSourceSection, splatTranslated, bodyTranslated);
                         rescueNodes.add(rescueNode);
                     } else {
-                        RubyNode result;
                         throw new UnsupportedOperationException();
                     }
                 } else {
@@ -3203,9 +3209,7 @@ protected RubyNode initFlipFlopStates(RubySourceSection sourceSection) {
 
     @Override
     protected RubyNode defaultVisit(ParseNode node) {
-        RubySourceSection sourceSection = translate(node.getPosition());
-        final RubyNode ret = nilNode(source, sourceSection);
-        return addNewlineIfNeeded(node, ret);
+        throw new UnsupportedOperationException(node.toString());
     }
 
     public TranslatorEnvironment getEnvironment() {