Patch from debian

[bastien-roucaries]

Hi,

We modified the generated regex script in order to rebuild at build time. Could you apply ?

subject: modify generate identifier in order to patch file
author: Bastien Roucariès <[email protected]>

Modify this script in order to use it in build rule: 

forwarded: yes
Index: newacorn/bin/generate-identifier-regex.js
===================================================================
--- newacorn.orig/bin/generate-identifier-regex.js
+++ newacorn/bin/generate-identifier-regex.js
@@ -51,7 +51,23 @@ function generate(chars) {
 
 var startData = generate(start), contData = generate(cont);
 
-console.log("let nonASCIIidentifierStartChars = \"" + startData.nonASCII + "\"");
-console.log("let nonASCIIidentifierChars = \"" + contData.nonASCII + "\"");
-console.log("const astralIdentifierStartCodes = " + JSON.stringify(startData.astral));
-console.log("const astralIdentifierCodes = " + JSON.stringify(contData.astral));
+
+
+if(process.argv.length != 3) {
+    console.log("let nonASCIIidentifierStartChars = \"" + startData.nonASCII + "\"");
+    console.log("let nonASCIIidentifierChars = \"" + contData.nonASCII + "\"");
+    console.log("const astralIdentifierStartCodes = " + JSON.stringify(startData.astral));
+    console.log("const astralIdentifierCodes = " + JSON.stringify(contData.astral));
+}
+else {
+    var fs = require('fs');
+    fs.readFile(process.argv[2], function(err, data) {
+    data.toString().split("\n").forEach(function(line) {
+       line = line.replace(/\s*let nonASCIIidentifierStartChars.*/, "let nonASCIIidentifierStartChars = \"" + startData.nonASCII + "\"");
+       line = line.replace(/\s*let nonASCIIidentifierChars.*/,"let nonASCIIidentifierChars = \"" + contData.nonASCII + "\"");
+        line = line.replace(/\s*const astralIdentifierStartCodes.*/,"const astralIdentifierStartCodes = " + JSON.stringify(startData.astral));
+        line = line.replace(/\s*const astralIdentifierCodes.*/,"const astralIdentifierCodes = " + JSON.stringify(contData.astral));
+       console.log(line);
+        });
+    });
+}

[bastien-roucaries]

we use it like node node bin/generate-identifier-regex.js src/identifier.js > build/src/identifer.js

[marijnh]

Could you clean this up so that it doesn't repeat the expressions? Ideally, put the lines in an array, and depending on the command line arguments, console.log them or generate a regexp from their first two words and replace on that.

[marijnh]

Guess not, then. I've applied a cleaned-up version of your proposed change in 3442a80