joepie91/pegjs
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

bin/pegjs: Default parser variable name is "module.exports"

Browse source 
The previous default name was "exports.parser". This meant that to use
the generated parser in Node.js, you had to use code like this:

  var parser = require("./my-cool-parser").parser;
  parser.parse(...);

Now you can shorten it a bit:

  var parser = require("./my-cool-parser");
  parser.parse(...);

The shorter version makes sense since no other objects except the parser
are exported from the module.
This commit is contained in:
David Majda 2011-02-13 12:43:12 +01:00
parent 13a3621e79
commit dcf904c392
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
bin/pegjs
View file

@ -22,7 +22,7 @@ function printHelp() {
sys.puts("");
sys.puts("Options:");
sys.puts(" -e, --export-var <variable> name of the variable where the parser object");
sys.puts(" will be stored (default: \"exports.parser\")");
sys.puts(" will be stored (default: \"module.exports\")");
sys.puts(" -v, --version print version information and exit");
sys.puts(" -h, --help print help and exit");
}
@ -63,7 +63,7 @@ function readStream(inputStream, callback) {
/* Main */
/* This makes the generated parser a CommonJS module by default. */
var exportVar = "exports.parser";
var exportVar = "module.exports";
while (args.length > 0 && isOption(args[0])) {
switch (args[0]) {