Browse Source

buffer: remove duplicate assertion tests

Many assertion tests are duplicated in buffer.js. These few could be easily
removed and still have all tests pass.
v0.9.4-release
Trevor Norris 12 years ago
committed by Ben Noordhuis
parent
commit
13c5db9771
  1. 33
      lib/buffer.js

33
lib/buffer.js

@ -1047,15 +1047,6 @@ Buffer.prototype.writeInt8 = function(value, offset, noAssert) {
var buffer = this; var buffer = this;
if (!noAssert) { if (!noAssert) {
assert.ok(value !== undefined && value !== null,
'missing value');
assert.ok(offset !== undefined && offset !== null,
'missing offset');
assert.ok(offset < buffer.length,
'Trying to write beyond buffer length');
verifsint(value, 0x7f, -0x80); verifsint(value, 0x7f, -0x80);
} }
@ -1068,18 +1059,6 @@ Buffer.prototype.writeInt8 = function(value, offset, noAssert) {
function writeInt16(buffer, value, offset, isBigEndian, noAssert) { function writeInt16(buffer, value, offset, isBigEndian, noAssert) {
if (!noAssert) { if (!noAssert) {
assert.ok(value !== undefined && value !== null,
'missing value');
assert.ok(typeof (isBigEndian) === 'boolean',
'missing or invalid endian');
assert.ok(offset !== undefined && offset !== null,
'missing offset');
assert.ok(offset + 1 < buffer.length,
'Trying to write beyond buffer length');
verifsint(value, 0x7fff, -0x8000); verifsint(value, 0x7fff, -0x8000);
} }
@ -1100,18 +1079,6 @@ Buffer.prototype.writeInt16BE = function(value, offset, noAssert) {
function writeInt32(buffer, value, offset, isBigEndian, noAssert) { function writeInt32(buffer, value, offset, isBigEndian, noAssert) {
if (!noAssert) { if (!noAssert) {
assert.ok(value !== undefined && value !== null,
'missing value');
assert.ok(typeof (isBigEndian) === 'boolean',
'missing or invalid endian');
assert.ok(offset !== undefined && offset !== null,
'missing offset');
assert.ok(offset + 3 < buffer.length,
'Trying to write beyond buffer length');
verifsint(value, 0x7fffffff, -0x80000000); verifsint(value, 0x7fffffff, -0x80000000);
} }

Loading…
Cancel
Save