Refactor comments

Author: wooorm

lib/constants.js

@@ -18,10 +18,10 @@ var formFeed = '\f'
 
 var whitespace = [space, tab, lineFeed, carriageReturn, formFeed]
 
-// https://html.spec.whatwg.org/#attribute-name-state
+// See: <https://html.spec.whatwg.org/#attribute-name-state>.
 var name = whitespace.concat(ampersand, slash, greaterThan, equalsTo)
 
-// https://html.spec.whatwg.org/#attribute-value-(unquoted)-state
+// See: <https://html.spec.whatwg.org/#attribute-value-(unquoted)-state>.
 var unquoted = whitespace.concat(ampersand, greaterThan)
 var unquotedSafe = unquoted.concat(
   nil,
@@ -32,13 +32,14 @@ var unquotedSafe = unquoted.concat(
   graveAccent
 )
 
-// https://html.spec.whatwg.org/#attribute-value-(single-quoted)-state
+// See: <https://html.spec.whatwg.org/#attribute-value-(single-quoted)-state>.
 var singleQuoted = [ampersand, apostrophe]
 
-// https://html.spec.whatwg.org/#attribute-value-(double-quoted)-state
+// See: <https://html.spec.whatwg.org/#attribute-value-(double-quoted)-state>.
 var doubleQuoted = [ampersand, quotationMark]
 
-// Maps of subsets. Each value is a matrix of tuples.
+// Maps of subsets.
+// Each value is a matrix of tuples.
 // The first value causes parse errors, the second is valid.
 // Of both values, the first value is unsafe, and the second is safe.
 module.exports = {

lib/element.js

@@ -55,8 +55,9 @@ function element(ctx, node, index, parent) {
   content = all(ctx, root)
 
   // If the node is categorised as void, but it has children, remove the
-  // categorisation.  This enables for example `menuitem`s, which are void in
-  // W3C HTML but not void in WHATWG HTML, to be stringified properly.
+  // categorisation.
+  // This enables for example `menuitem`s, which are void in W3C HTML but not
+  // void in WHATWG HTML, to be stringified properly.
   selfClosing = content ? false : selfClosing
 
   if (attrs || !omit || !omit.opening(node, index, parent)) {

lib/omission/opening.js

@@ -65,9 +65,10 @@ function body(node) {
   )
 }
 
-// Whether to omit `<colgroup>`.  The spec describes some logic for the opening
-// tag, but it’s easier to implement in the closing tag, to the same effect, so
-// we handle it there instead.
+// Whether to omit `<colgroup>`.
+// The spec describes some logic for the opening tag, but it’s easier to
+// implement in the closing tag, to the same effect, so we handle it there
+// instead.
 function colgroup(node, index, parent) {
   var prev = before(parent, index)
   var head = first(node, true)

test/omission-closing-menuitem.js

@@ -43,9 +43,8 @@ test('`menuitem` (closing)', function(t) {
     'should not omit tag followed by others'
   )
 
-  /* This actually tests an edge case where `menuitems`,
-   * which can have children in WHATWG HTML, but not in
-   * W3C HTML, here do not have children. */
+  // This actually tests an edge case where `menuitems`, which can have children
+  // in WHATWG HTML, but not in W3C HTML, here do not have children.
   t.deepEqual(
     to(h('menu', [h('menuitem'), h('p')]), {omitOptionalTags: true}),
     '<menu><menuitem><p></menu>',