Simplify switch statement (#550)

Both the 'undefined' and 'function' cases can be handled in the same way.
pinojs · Nov 22, 2018 · 155d75e · 155d75e
1 parent 3b7019e
commit 155d75e
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/lib/tools.js b/lib/tools.js
@@ -108,7 +108,9 @@ function asJson (obj, msg, num, time) {
         value = serializers[key] ? serializers[key](value) : value
 
         switch (typeof value) {
-          case 'undefined': continue
+          case 'undefined':
+          case 'function':
+            continue
           case 'number':
             /* eslint no-fallthrough: "off" */
             if (Number.isFinite(value) === false) {
@@ -122,10 +124,6 @@ function asJson (obj, msg, num, time) {
           case 'string':
             value = (stringifiers[key] || asString)(value)
             break
-          case 'function':
-            // ignore functions
-            value = undefined
-            break
           default:
             value = (stringifiers[key] || stringify)(value)
         }