thriftrw - npm Package Compare versions

Comparing version 2.3.0 to 2.3.1

package.json

 {
   "name": "thriftrw",
-  "version": "2.3.0",
+  "version": "2.3.1",
   "description": "thrift encoding/decoding using bufrw",
@@ -5,0 +5,0 @@ "keywords": [],

skip.js

@@ -237,1 +237,2 @@ // Copyright (c) 2015 Uber Technologies, Inc.
 module.exports.skipField = skipField;
+module.exports.skipType = skipType;

struct.js

@@ -31,3 +31,3 @@ // Copyright (c) 2015 Uber Technologies, Inc.
 var errors = require('./errors');
-var skipField = require('./skip').skipField;
+var skipType = require('./skip').skipType;
 
@@ -376,3 +376,3 @@ var LengthResult = bufrw.LengthResult;
         if (!self.spec.fieldsById[id]) {
-            result = skipField(buffer, offset);
+            result = skipType(buffer, offset, typeid);
             // istanbul ignore if
@@ -379,0 +379,0 @@ if (result.err) {

test/struct-skip.js

@@ -38,3 +38,4 @@ // Copyright (c) 2015 Uber Technologies, Inc.
         0x00, 0x02,               // id:2     -- 2 -- WHAT EVEN IS!?
-        0x01,                     // typeid:1 -- 1 -- VOID
+        0x00,                     // bool:1
+
         0x00                      // typeid:1 -- 0 -- STOP
@@ -51,5 +52,4 @@ ]), 0);
     var result = Health.rw.readFrom(new Buffer([
-        0x02,                     // type:1   -- 2  -- BOOL
+        11,                       // typeid:1 -- 11 -- STRING
         0x00, 0x02,               // id:2     -- 2  -- WHAT EVEN IS!?
-        11,                       // typeid:1 -- 11 -- STRING
         0x00, 0x00, 0x00, 0x02,   // len~4
@@ -68,12 +68,9 @@ 0x20, 0x20,               // '  '
     var result = Health.rw.readFrom(new Buffer([
-        0x02,                     // type:1   -- 2  -- BOOL
-        0x00, 0x02,               // id:2     -- 2  -- WHAT EVEN IS!?
         12,                       // typeid:1 -- 12 -- STRUCT
-        0x01,                     // typeid:1 -- 1  -- VOID
-        0x01,                     // typeid:1 -- 1  -- VOID
-        11,                       // typeid:1 -- 11 -- STRING
-        0x00, 0x00, 0x00, 0x02,   // len~4
-        0x20, 0x20,               // '  '
-        0x01,                     // typeid:1 -- 1  -- VOID
-        0x00,                     // typeid:1 -- 0  -- STOP
+        0x00, 0x02,               // id:2     -- 2  -- ?
+        11,                       //   typeid:1 -- 11 -- STRING
+        0x00, 0x01,               //   fieldid:2 -- 1 -- ?
+        0x00, 0x00, 0x00, 0x02,   //   len~4
+        0x20, 0x20,               //   '  '
+        0x00,                     //   typeid:1 -- 0  -- STOP
         0x00                      // typeid:1 -- 0  -- STOP
@@ -90,5 +87,4 @@ ]), 0);
     var result = Health.rw.readFrom(new Buffer([
-        0x02,                     // type:1           -- 2 BOOL
-        0x00, 0x02,               // id:2             -- 2 UNKNOWN
         0x0d,                   // typeid:1           -- 13, map
+        0x00, 0x02,             // id:2               -- 2 UNKNOWN
 
@@ -95,0 +91,0 @@ // Thus begins a large map

test/struct.js

@@ -97,6 +97,8 @@ // Copyright (c) 2015 Uber Technologies, Inc.
         0x00, 0x02,               // id:2     -- 2 -- WHAT EVEN IS!?
-        0x01,                     // typeid:1 -- 1 -- VOID
+        0x00,                     // bool:1
+
         0x02,                     // type:1   -- 2 -- BOOL
         0x00, 0x01,               // id:2     -- 1 -- ok
         0x01,                     // ok:1     -- 1 -- true
+
         0x00                      // typeid:1 -- 0 -- STOP
@@ -132,7 +134,8 @@ ]), 0);
         0x01,                     // ok:1     -- 1 -- true
-        0x02,                     // type:1   -- 2  -- BOOL
+
+        11,                       // typeid:1 -- 11 -- STRING
         0x00, 0x02,               // id:2     -- 2  -- WHAT EVEN IS!?
-        11,                       // typeid:1 -- 11 -- STRING
         0x00, 0x00, 0x00, 0x02,   // len~4
         0x20, 0x20,               // '  '
+
         0x00                      // typeid:1 -- 0  -- STOP
@@ -152,5 +155,6 @@ ]), 0);
         0x01,                     // ok:1     -- 1 -- true
-        0x02,                     // type:1   -- 2  -- BOOL
+
+        0x0c,                     // typeid:1 -- 12 -- STRUCT
         0x00, 0x02,               // id:2     -- 2  -- WHAT EVEN IS!?
-        0x0c,                     // typeid:1 -- 12 -- STRUCT
+
         0x01,                     // typeid:1 -- 1  -- VOID
@@ -163,2 +167,3 @@ 0x01,                     // typeid:1 -- 1  -- VOID
         0x00,                     // typeid:1 -- 0  -- STOP
+
         0x00                      // typeid:1 -- 0  -- STOP
@@ -179,5 +184,4 @@ ]), 0);
 
-        0x02,                     // type:1   -- 2 BOOL
+        0x0d,                     // typeid:1 -- 13, map
         0x00, 0x02,               // id:2     -- 2 UNKNOWN
-        0x0d,                     // typeid:1 -- 13, map
 
@@ -239,5 +243,4 @@ // Thus begins a large map
 
-        0x02,                     // type:1      -- 2 BOOL
+        0x0f,                     // typeid:1    -- 15, list
         0x00, 0x02,               // id:2        -- 2 UNKNOWN
-        0x0f,                     // typeid:1    -- 15, list
 
@@ -244,0 +247,0 @@ // Thus begins a list

No alert changes

Worsened metrics

Total package byte prevSize

356708

decreased by-0.15%

Lines of code

6238

decreased by-0.13%

No dependency changes