some comments

arangod/Aql/AqlValue.h

@@ -269,6 +269,9 @@ namespace std {
         case triagens::aql::AqlValue::RANGE: {
           return a._range == b._range;
         }
+        // case triagens::aql::AqlValue::EMPTY intentionally not handled here!
+        // (should fall through and fail!)
+
         default: {
           TRI_ASSERT(false);
           return true;

arangod/Aql/V8Executor.cpp

@@ -851,7 +851,8 @@ void V8Executor::generateCodeNode (AstNode const* node) {
       break;
 
     case NODE_TYPE_VARIABLE:
-      // we're not expecting a variable here
+    case NODE_TYPE_PARAMETER:
+      // we're not expecting these types here
       THROW_ARANGO_EXCEPTION(TRI_ERROR_INTERNAL);
 
     default: