skip encoding null values altogether (#43)

Author: mourner

index.js

@@ -94,17 +94,20 @@ function writeProperties (context, pbf) {
   var valuecache = context.valuecache
 
   for (var key in feature.properties) {
+    var value = feature.properties[key]
+
     var keyIndex = keycache[key]
+    if (value === null) continue // don't encode null value properties
+
     if (typeof keyIndex === 'undefined') {
       keys.push(key)
       keyIndex = keys.length - 1
       keycache[key] = keyIndex
     }
     pbf.writeVarint(keyIndex)
 
-    var value = feature.properties[key]
     var type = typeof value
-    if (value !== null && type !== 'string' && type !== 'boolean' && type !== 'number') {
+    if (type !== 'string' && type !== 'boolean' && type !== 'number') {
       value = JSON.stringify(value)
     }
     var valueKey = type + ':' + value

test/properties.js

@@ -55,7 +55,7 @@ test('property encoding', function (t) {
     var second = layer.feature(1).properties
     t.same(first.c, '{"hello":"world"}')
     t.same(first.d, '[1,2,3]')
-    t.equals(first.e, null)
+    t.equals(first.e, undefined)
     t.same(second.c, '{"goodbye":"planet"}')
     t.same(second.d, '{"hello":"world"}')
     t.end()