Pull options out from param setup to allow easier extension.

This commit is contained in:
Blake Embrey
2013-12-28 22:31:45 -05:00
parent 6c2137a420
commit 150e55aa00
+10 -5
View File
@@ -798,9 +798,7 @@ JavaScriptCompiler.prototype = {
};
},
// the params and contexts arguments are passed in arrays
// to fill in
setupParams: function(paramSize, params, useRegister) {
setupOptions: function(paramSize, params) {
var options = [], contexts = [], types = [], param, inverse, program;
options.push("hash:" + this.popStack());
@@ -846,14 +844,21 @@ JavaScriptCompiler.prototype = {
options.push("data:data");
}
options = "{" + options.join(",") + "}";
return options;
},
// the params and contexts arguments are passed in arrays
// to fill in
setupParams: function(paramSize, params, useRegister) {
var options = '{' + this.setupOptions(paramSize, params).join(',') + '}';
if (useRegister) {
this.register('options', options);
params.push('options');
} else {
params.push(options);
}
return params.join(", ");
return params.join(', ');
}
};