Refactor style

Author: wooorm

index.js

@@ -43,8 +43,7 @@ function toMdast(tree, options) {
     return augment(node, result)
   }
 
-  /* `right` is the finalized mdast node,
-   * created from `left`, a hast node */
+  // `right` is the finalized mdast node, created from `left`, a hast node.
   function augment(left, right) {
     if (left.position) {
       right.position = left.position

lib/handlers/list-item.js

@@ -13,7 +13,7 @@ function listItem(h, node) {
   var grandchildren
   var content
 
-  /* Check if this node starts with a checkbox. */
+  // Check if this node starts with a checkbox.
   if (head && is(head, 'p')) {
     grandchildren = head.children
     checkbox = grandchildren[0]
@@ -29,13 +29,13 @@ function listItem(h, node) {
 
   content = wrapChildren(h, node)
 
-  /* Remove initial spacing if we previously found a checkbox. */
+  // Remove initial spacing if we previously found a checkbox.
   if (checked !== null) {
     grandchildren = content[0] && content[0].children
     head = grandchildren && grandchildren[0]
 
     if (head && head.type === 'text' && head.value.charAt(0) === ' ') {
-      /* Remove text with one space, or remove that one initial space */
+      // Remove text with one space, or remove that one initial space.
       if (head.value.length === 1) {
         content[0].children = grandchildren.slice(1)
       } else {
@@ -44,7 +44,7 @@ function listItem(h, node) {
     }
   }
 
-  /* Drop empty paragraph. */
+  // Drop empty paragraph.
   if (content.length === 1) {
     head = content[0]
 

lib/handlers/media.js

@@ -30,7 +30,7 @@ function media(h, node) {
 
   visit({type: 'root', children: nodes}, 'link', findLink)
 
-  /* If the content links to something, or if it’s not phrasing... */
+  // If the content links to something, or if it’s not phrasing...
   if (linkInFallbackContent || needed(nodes)) {
     return nodes
   }

lib/handlers/table.js

@@ -10,7 +10,7 @@ function table(h, node) {
   return h(node, 'table', {align: align}, patch(all(h, node), align.length))
 }
 
-/* Infer the alignment of the table. */
+// Infer the alignment of the table.
 function alignment(node) {
   var align = []
 
@@ -30,7 +30,7 @@ function alignment(node) {
   }
 }
 
-/* Count cells in `parent` before `node`. */
+// Count cells in `parent` before `node`.
 function cellsBefore(parent, node) {
   var children = parent.children
   var length = children.length
@@ -54,12 +54,12 @@ function cellsBefore(parent, node) {
   return pos
 }
 
-/* Check if `node` is a cell. */
+// Check if `node` is a cell.
 function cell(node) {
   return node.tagName === 'th' || node.tagName === 'td'
 }
 
-/* Ensure the amount of cells in a row matches `align.left`. */
+// Ensure the amount of cells in a row matches `align.left`.
 function patch(rows, count) {
   var length = rows.length
   var index = -1

lib/util/wrap.js

@@ -6,7 +6,7 @@ wrap.needed = needed
 
 var phrasing = require('mdast-util-phrasing')
 
-/* Wrap all inline runs of mdast content in `paragraph` nodes. */
+// Wrap all inline runs of mdast content in `paragraph` nodes.
 function wrap(nodes) {
   var result = []
   var length = nodes.length
@@ -48,9 +48,9 @@ function wrap(nodes) {
   }
 }
 
-/* Check if there are non-inline mdast nodes returned.
- * This is needed if a fragment is given, which could just be
- * a sentence, and doesn’t need a wrapper paragraph. */
+// Check if there are non-inline mdast nodes returned.  This is needed if a
+// fragment is given, which could just be a sentence, and doesn’t need a
+// wrapper paragraph.
 function needed(nodes) {
   var length = nodes.length
   var index = -1

test/index.js

@@ -123,7 +123,7 @@ test('fixtures', function(t) {
 
       var tree = remove(fromHTML.runSync(fromHTML.parse(input)), true)
 
-      /* Replace middots with spaces (useful for break nodes). */
+      // Replace middots with spaces (useful for break nodes).
       output = output.replace(/·/g, ' ')
 
       st.doesNotThrow(function() {