Refactor code-style

Author: wooorm

lib/index.js

@@ -1,15 +1,12 @@
 'use strict'
 
-/* Dependencies. */
 var xtend = require('xtend')
 var defaults = require('./github.json')
 
-/* Expose. */
 module.exports = wrapper
 
 var own = {}.hasOwnProperty
 
-/* Schema. */
 var NODES = {
   root: {children: all},
   doctype: handleDoctype,
@@ -26,7 +23,7 @@ var NODES = {
   }
 }
 
-/* Sanitize `node`, according to `schema`. */
+// Sanitize `node`, according to `schema`.
 function wrapper(node, schema) {
   var ctx = {type: 'root', children: []}
   var replace
@@ -54,7 +51,7 @@ function wrapper(node, schema) {
   return replace
 }
 
-/* Sanitize `node`. */
+// Sanitize `node`.
 function one(schema, node, stack) {
   var type = node && node.type
   var replacement = {type: node.type}
@@ -84,7 +81,7 @@ function one(schema, node, stack) {
         if (result === false) {
           replace = false
 
-          /* Set the non-safe value. */
+          // Set the non-safe value.
           replacement[key] = node[key]
         } else if (result !== null && result !== undefined) {
           replacement[key] = result
@@ -108,7 +105,7 @@ function one(schema, node, stack) {
   return replacement
 }
 
-/* Sanitize `children`. */
+// Sanitize `children`.
 function all(schema, children, node, stack) {
   var nodes = children || []
   var length = nodes.length || 0
@@ -133,7 +130,7 @@ function all(schema, children, node, stack) {
   return results
 }
 
-/* Sanitize `properties`. */
+// Sanitize `properties`.
 function handleProperties(schema, properties, node, stack) {
   var name = handleTagName(schema, node.tagName, node, stack)
   var attrs = schema.attributes
@@ -170,7 +167,7 @@ function handleProperties(schema, properties, node, stack) {
   return result
 }
 
-/* Sanitize a property value which is a list. */
+// Sanitize a property value which is a list.
 function handlePropertyValues(schema, values, prop) {
   var length = values.length
   var result = []
@@ -188,7 +185,7 @@ function handlePropertyValues(schema, values, prop) {
   return result
 }
 
-/* Sanitize a property value. */
+// Sanitize a property value.
 function handlePropertyValue(schema, value, prop) {
   if (
     typeof value !== 'boolean' &&
@@ -209,7 +206,7 @@ function handlePropertyValue(schema, value, prop) {
   return value
 }
 
-/* Check whether `value` is a safe URL. */
+// Check whether `value` is a safe URL.
 function handleProtocol(schema, value, prop) {
   var protocols = schema.protocols
   var protocol
@@ -266,12 +263,12 @@ function handleProtocol(schema, value, prop) {
   return false
 }
 
-/* Always return a valid HTML5 doctype. */
+// Always return a valid HTML5 doctype.
 function handleDoctypeName() {
   return 'html'
 }
 
-/* Sanitize `tagName`. */
+// Sanitize `tagName`.
 function handleTagName(schema, tagName, node, stack) {
   var name = typeof tagName === 'string' ? tagName : null
   var ancestors = schema.ancestors
@@ -284,8 +281,8 @@ function handleTagName(schema, tagName, node, stack) {
 
   ancestors = own.call(ancestors, name) ? ancestors[name] : []
 
-  /* Some nodes can break out of their context if they
-   * don’t have a certain ancestor. */
+  // Some nodes can break out of their context if they don’t have a certain
+  // ancestor.
   if (ancestors.length !== 0) {
     length = ancestors.length + 1
     index = -1
@@ -312,17 +309,17 @@ function handleComment(schema) {
   return schema.allowComments ? {value: handleValue} : null
 }
 
-/* Sanitize `value`. */
+// Sanitize `value`.
 function handleValue(schema, value) {
   return typeof value === 'string' ? value : ''
 }
 
-/* Allow `value`. */
+// Allow `value`.
 function allow(schema, value) {
   return value
 }
 
-/* Check if `prop` is a data property. */
+// Check if `prop` is a data property.
 function data(prop) {
   return prop.length > 4 && prop.slice(0, 4).toLowerCase() === 'data'
 }

test.js

@@ -1,6 +1,5 @@
 'use strict'
 
-/* Dependencies. */
 var test = require('tape')
 var html = require('hast-util-to-html')
 var h = require('hastscript')
@@ -12,7 +11,6 @@ var sanitize = require('.')
 
 /* eslint-disable no-script-url, max-params */
 
-/* Tests. */
 test('sanitize()', function(t) {
   t.test('non-node', function(st) {
     st.equal(html(sanitize(true)), '', 'should ignore non-nodes (#1)')
@@ -525,21 +523,21 @@ test('sanitize()', function(t) {
   t.end()
 })
 
-/* Coverage. */
+// Coverage.
 toString()
 
-/* Check */
+// Check.
 function toString() {
   return 'alert(1);'
 }
 
-/* Test `valid` and `invalid` `url`s in `prop` on `tagName`. */
+// Test `valid` and `invalid` `url`s in `prop` on `tagName`.
 function testAllURLs(t, tagName, prop, all) {
   testURLs(t, tagName, prop, all.valid, true)
   testURLs(t, tagName, prop, all.invalid, false)
 }
 
-/* Test `valid` `url`s in `prop` on `tagName`. */
+// Test `valid` `url`s in `prop` on `tagName`.
 function testURLs(t, tagName, prop, urls, valid) {
   Object.keys(urls).forEach(function(name) {
     var props = {}