improved version of SHA-1: 1cb19b4 (:: Fix performance bug: Dynamic argumets - Bad value context for argument value): as we know the %parse-param names of each extra parse call parameter, we can use those to create a customized version of the parser for the given grammar + param set, without resorting slow slice(arguments) code or (also slow) bind+function wrap.

Author: GerHobbelt

lib/jison.js

@@ -2916,6 +2916,14 @@ lrGeneratorMixin.generateModule_ = function generateModule_() {
 
     parseFn = removeUnusedKernelFeatures(parseFn, this);
 
+    // fill in the optional, extra parse parameters (`%parse-param ...`)
+    // in the generated parser:
+    if (!this.parseParams) {
+        parseFn = parseFn.replace(/, parseParams/g, '');
+    } else {
+        parseFn = parseFn.replace(/, parseParams/g, ', ' + this.parseParams.join(', '));
+    }
+
     var errorClassCode = this.generateErrorClass();
 
     // set up the 'option' `exportAllTables` as a hash object for returning
@@ -4391,7 +4399,7 @@ for (var api in api_set) {
 }
 
 
-parser.parse = function parse(input) {
+parser.parse = function parse(input, parseParams) {
     var self = this,
         stack = new Array(128),         // token stack: stores token which leads to state at the same index (column storage)
         sstack = new Array(128),        // state stack: stores states
@@ -4407,8 +4415,6 @@ parser.parse = function parse(input) {
         ERROR_RECOVERY_TOKEN_DISCARD_COUNT = (this.options.errorRecoveryTokenDiscardCount | 0) || 3;
     var NO_ACTION = [0, table.length /* ensures that anyone using this new state will fail dramatically! */];
 
-    var args = stack.slice.call(arguments, 1);
-
     //this.reductionCount = this.shiftCount = 0;
 
     var lexer;
@@ -4583,11 +4589,11 @@ parser.parse = function parse(input) {
 
         if (invoke_post_methods) {
             if (sharedState.yy.post_parse) {
-                rv = sharedState.yy.post_parse.apply(this, [sharedState.yy, resultValue].concat(args));
+                rv = sharedState.yy.post_parse.apply(this, [sharedState.yy, resultValue, parseParams]);
                 if (typeof rv !== 'undefined') resultValue = rv;
             }
             if (this.post_parse) {
-                rv = this.post_parse.apply(this, [sharedState.yy, resultValue].concat(args));
+                rv = this.post_parse.apply(this, [sharedState.yy, resultValue, parseParams]);
                 if (typeof rv !== 'undefined') resultValue = rv;
             }
         }
@@ -4740,10 +4746,10 @@ _handle_error_end_of_section:                   // this concludes the error reco
         this.__reentrant_call_depth++;
 
         if (this.pre_parse) {
-            this.pre_parse.apply(this, [sharedState.yy].concat(args));
+            this.pre_parse.apply(this, [sharedState.yy, parseParams]);
         }
         if (sharedState.yy.pre_parse) {
-            sharedState.yy.pre_parse.apply(this, [sharedState.yy].concat(args));
+            sharedState.yy.pre_parse.apply(this, [sharedState.yy, parseParams]);
         }
 
         newState = sstack[sp - 1];
@@ -4967,7 +4973,7 @@ _handle_error_end_of_section:                  // this concludes the error recov
                   yyval._$.range = [lstack[lstack_begin].range[0], lstack[lstack_end].range[1]];
                 }
 
-                r = this.performAction.apply(yyval, [yytext, yyleng, yylineno, yyloc, sharedState.yy, newState, sp - 1, vstack, lstack, stack, sstack].concat(args));
+                r = this.performAction.apply(yyval, [yytext, yyleng, yylineno, yyloc, sharedState.yy, newState, sp - 1, vstack, lstack, stack, sstack, parseParams]);
 
                 if (typeof r !== 'undefined') {
                     retval = r;

lib/util/ebnf-parser.js

@@ -25,9 +25,9 @@ bnf.yy.addDeclaration = function bnfAddDeclaration(grammar, decl) {
         decl.token_list.forEach(function (tok) {
             grammar.extra_tokens.push(tok);
         });
-    } else if (decl.parseParam) {
+    } else if (decl.parseParams) {
         if (!grammar.parseParams) grammar.parseParams = [];
-        grammar.parseParams = grammar.parseParams.concat(decl.parseParam);
+        grammar.parseParams = grammar.parseParams.concat(decl.parseParams);
     } else if (decl.parserType) {
         if (!grammar.options) grammar.options = {};
         grammar.options.type = decl.parserType;