smalloc: don't allow to dispose typed arrays

[vkurchatkin]

Typed arrays behave as objects with external data in some cases, but disposing them is a bad idea:

var smalloc = require('smalloc');

var ab = new ArrayBuffer(3);
var a1 = new Uint8Array(ab);
var a2 = new Uint8Array(ab);

for (var i = 0; i < a1.length; i++)
  a1[i] = 6;

console.log(a1);
console.log(a2);

smalloc.dispose(a1);

console.log(a1);
console.log(a2);

// node(72543,0x7fff7ada2310) malloc: *** error for object 0x100f0b2f0: pointer being freed was not allocated
// *** set a breakpoint in malloc_error_break to debug
// Abort trap: 6

/cc @trevnorris

[trevnorris]

Nice catch. Does Value::IsTypedArray() also catch DataView?

[vkurchatkin]

@trevnorris it doesn't, but Value::IsArrayBufferView does. smalloc.hasExternalData returns false for DataView, so it's not necessary

[vkurchatkin]

@trevnorris ping

[trevnorris]

Thanks much. Landed in bf3e0f4.