switched to proper Set and Map instances

replacing poor man's versions

in the process, also got rid of obsolete workarounds for legacy versions

Author: FND

lib/util/files/index.js

@@ -1,40 +1,28 @@
 "use strict";
 
-let { abort, repr } = require("../");
 let { writeFile } = require("fs/promises");
 let { mkdirSync } = require("fs");
 let path = require("path");
 
-let KNOWN = {}; // avoids redundant `mkdirp` invocations
-let LOCKS = {};
+let KNOWN = new Set(); // avoids redundant `mkdir` invocations
+let LOCKS = new Map();
 
 // avoids concurrent write operations and creates target directory if necessary
 module.exports = function createFile(filepath, contents) {
-	let lock = LOCKS[filepath];
+	let lock = LOCKS.get(filepath);
 	if(lock) { // defer
 		return lock.then(_ => createFile(filepath, contents));
 	}
 
 	// create directory if necessary
-	if(!KNOWN[filepath]) {
-		KNOWN[filepath] = true;
-		// NB: `sync` avoids race condition for subsequent operations
-		mkdirpSync(path.dirname(filepath));
+	if(!KNOWN.has(filepath)) {
+		KNOWN.add(filepath);
+		mkdirSync(path.dirname(filepath), { recursive: true });
 	}
 
 	let prom = writeFile(filepath, contents);
-	LOCKS[filepath] = prom;
+	LOCKS.set(filepath, prom);
 	return prom.then(_ => {
-		delete LOCKS[filepath];
+		LOCKS.delete(filepath);
 	});
 };
-
-function mkdirpSync(directory) {
-	try {
-		// NB: `recursive` option was introduced in Node v10.12.0
-		mkdirSync(directory, { recursive: true });
-	} catch(err) {
-		abort(`ERROR: auto-creating ${repr(directory)} requires ` +
-				"Node v10.12.0 or above");
-	}
-}