Refactor comments

Author: wooorm

lib/handlers/dl.js

@@ -43,7 +43,7 @@ function dataList(h, node) {
 
   groups.push(group)
 
-  // Group titles and definitions.
+  // Create items.
   length = groups.length
   index = -1
   content = []

lib/handlers/iframe.js

@@ -9,9 +9,9 @@ function iframe(h, node) {
   var title = node.properties.title
 
   // Only create a link if there is a title.
-  // We can’t use `content` because conforming HTML parsers treat it as text,
-  // whereas legacy parsers treat it as HTML, so they will likely contain tags
-  // that will show up in text.
+  // We can’t use the content of the frame because conforming HTML parsers treat
+  // it as text, whereas legacy parsers treat it as HTML, so it will likely
+  // contain tags that will show up in text.
   if (src && title) {
     return {
       type: 'link',

lib/handlers/media.js

@@ -31,7 +31,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/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 runs of mdast phrasing 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-phrasing 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