Skip to content

Commit 3309514

Browse files
committed
fix int8 range in encoding
pointed by msgpack/website#19
1 parent 3d9faef commit 3309514

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

src/Encoder.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ export class Encoder {
9999
if (object >= -0x20) {
100100
// nagative fixint
101101
this.writeU8(0xe0 | (object + 0x20));
102-
} else if (object > -0x80) {
102+
} else if (object >= -0x80) {
103103
// int 8
104104
this.writeU8(0xd0);
105105
this.writeI8(object);

test/msgpack-test-suite.test.ts

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -120,4 +120,10 @@ describe("msgpack-test-suite", () => {
120120
});
121121
}
122122
});
123+
124+
describe("encoding in minimum values", () => {
125+
it("int 8", () => {
126+
assert.deepStrictEqual(encode(-128), Uint8Array.from([0xd0, 0x80]));
127+
});
128+
});
123129
});

0 commit comments

Comments
 (0)