Refactor code-style

Author: wooorm

.editorconfig

@@ -0,0 +1,9 @@
+root = true
+
+[*]
+indent_style = space
+indent_size = 2
+end_of_line = lf
+charset = utf-8
+trim_trailing_whitespace = true
+insert_final_newline = true

index.js

@@ -1,44 +1,62 @@
-'use strict';
+'use strict'
 
-var is = require('unist-util-is');
+var is = require('unist-util-is')
 
+module.exports = remove
 
-module.exports = function (ast, opts, predicate) {
-  if (arguments.length == 2) {
-    predicate = opts;
-    opts = {};
+function remove(ast, opts, test) {
+  var cascade
+
+  if (!test) {
+    test = opts
+    opts = {}
   }
 
-  opts.cascade = opts.cascade || opts.cascade === undefined;
+  cascade = opts.cascade
+  cascade = cascade === null || cascade === undefined ? true : cascade
+
+  return preorder(ast, null, null)
 
   // Check and remove nodes recursively in preorder.
   // For each composite node, modify its children array in-place.
-  return (function preorder (node, nodeIndex, parent) {
-    if (is(predicate, node, nodeIndex, parent)) {
-      return null;
+  function preorder(node, nodeIndex, parent) {
+    var children
+    var length
+    var index
+    var position
+    var child
+
+    if (is(test, node, nodeIndex, parent)) {
+      return null
     }
-    if (!node.children || !node.children.length) {
-      return node;
+
+    children = node.children
+
+    if (!children || children.length === 0) {
+      return node
     }
 
     // Move all living children to the beginning of the children array.
+    position = 0
+    length = children.length
+    index = -1
 
-    var length = 0;
-
-    for (var index = 0; index < node.children.length; ++index) {
-      var child = preorder(node.children[index], index, node);
+    while (++index < length) {
+      child = preorder(children[index], index, node)
 
       if (child) {
-        node.children[length++] = child;
+        children[position++] = child
       }
     }
 
-    if (!length && opts.cascade) {
-      // Cascade delete.
-      return null;
+    // Cascade delete.
+    if (cascade && position === 0) {
+      return null
     }
 
-    node.children.length = length;
-    return node;
-  }(ast, null, null));
-};
+    // Drop other nodes.
+    children.length = position
+
+    return node
+  }
+}

package.json

@@ -30,15 +30,34 @@
     "unist-util-is": "^2.0.0"
   },
   "devDependencies": {
+    "prettier": "^1.13.3",
     "remark-cli": "^5.0.0",
     "remark-preset-wooorm": "^4.0.0",
     "tape": "^4.4.0",
-    "unist-builder": "^1.0.0"
+    "unist-builder": "^1.0.0",
+    "xo": "^0.21.1"
   },
   "scripts": {
-    "format": "remark . -qfo",
-    "test": "tape test/*.js",
-    "cov": "nyc --reporter lcov tape test/*.js"
+    "format": "remark . -qfo && prettier --write '**/*.js' && xo --fix",
+    "test": "node test",
+    "cov": "nyc --reporter lcov tape test.js"
+  },
+  "prettier": {
+    "tabWidth": 2,
+    "useTabs": false,
+    "singleQuote": true,
+    "bracketSpacing": false,
+    "semi": false,
+    "trailingComma": "none"
+  },
+  "xo": {
+    "prettier": true,
+    "esnext": false,
+    "rules": {
+      "guard-for-in": "off",
+      "no-var": "off",
+      "prefer-arrow-callback": "off"
+    }
   },
   "remarkConfig": {
     "plugins": [