From: Ben Noordhuis Date: Wed, 29 Apr 2015 17:08:19 +0000 (+0200) Subject: lib: deprecate the smalloc module X-Git-Tag: v2.0.0~42 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d5ce47e433c24e9773373ccede613df9f051e7b1;p=platform%2Fupstream%2Fnodejs.git lib: deprecate the smalloc module Upcoming V8 changes will make it impossible to keep supporting the smalloc module so deprecate it now and tell people to switch to typed arrays. The module is used in core in a few places so this commit makes the public module private and replaces the public part with wrappers that print deprecation notices. PR-URL: https://github.com/iojs/io.js/pull/1564 Reviewed-By: Domenic Denicola Reviewed-By: Trevor Norris --- diff --git a/lib/internal/smalloc.js b/lib/internal/smalloc.js new file mode 100644 index 0000000..f181cb6 --- /dev/null +++ b/lib/internal/smalloc.js @@ -0,0 +1,91 @@ +'use strict'; + +const smalloc = process.binding('smalloc'); +const kMaxLength = smalloc.kMaxLength; +const kMinType = smalloc.kMinType; +const kMaxType = smalloc.kMaxType; +const util = require('util'); + +exports.alloc = alloc; +exports.copyOnto = copyOnto; +exports.dispose = dispose; +exports.hasExternalData = hasExternalData; + +// don't allow kMaxLength to accidentally be overwritten. it's a lot less +// apparent when a primitive is accidentally changed. +Object.defineProperty(exports, 'kMaxLength', { + enumerable: true, value: kMaxLength, writable: false +}); + +Object.defineProperty(exports, 'Types', { + enumerable: true, value: Object.freeze(smalloc.types), writable: false +}); + + +// usage: obj = alloc(n[, obj][, type]); +function alloc(n, obj, type) { + n = n >>> 0; + + if (obj === undefined) + obj = {}; + + if (typeof obj === 'number') { + type = obj >>> 0; + obj = {}; + } else if (util.isPrimitive(obj)) { + throw new TypeError('obj must be an Object'); + } + + if (Array.isArray(obj)) + throw new TypeError('obj cannot be an array'); + if (obj instanceof Buffer) + throw new TypeError('obj cannot be a Buffer'); + if (smalloc.isTypedArray(obj)) + throw new TypeError('obj cannot be a typed array'); + if (smalloc.hasExternalData(obj)) + throw new TypeError('object already has external array data'); + + if (type < kMinType || type > kMaxType) + throw new TypeError('unknown external array type: ' + type); + if (n > kMaxLength) + throw new RangeError('Attempt to allocate array larger than maximum ' + + 'size: 0x' + kMaxLength.toString(16) + ' elements'); + + return smalloc.alloc(obj, n, type); +} + + +function dispose(obj) { + if (util.isPrimitive(obj)) + throw new TypeError('obj must be an Object'); + if (obj instanceof Buffer) + throw new TypeError('obj cannot be a Buffer'); + if (smalloc.isTypedArray(obj)) + throw new TypeError('obj cannot be a typed array'); + if (!smalloc.hasExternalData(obj)) + throw new TypeError('obj has no external array data'); + + smalloc.dispose(obj); +} + + +function copyOnto(source, sourceStart, dest, destStart, copyLength) { + if (util.isPrimitive(source)) + throw new TypeError('source must be an Object'); + if (util.isPrimitive(dest)) + throw new TypeError('dest must be an Object'); + if (!smalloc.hasExternalData(source)) + throw new TypeError('source has no external array data'); + if (!smalloc.hasExternalData(dest)) + throw new TypeError('dest has no external array data'); + + return smalloc.copyOnto(source, sourceStart, dest, destStart, copyLength); +} + + +function hasExternalData(obj) { + if (util.isPrimitive(obj)) + return false; + + return smalloc.hasExternalData(obj); +} diff --git a/lib/smalloc.js b/lib/smalloc.js index f181cb6..8601a8b 100644 --- a/lib/smalloc.js +++ b/lib/smalloc.js @@ -1,91 +1,27 @@ 'use strict'; -const smalloc = process.binding('smalloc'); -const kMaxLength = smalloc.kMaxLength; -const kMinType = smalloc.kMinType; -const kMaxType = smalloc.kMaxType; -const util = require('util'); +const smalloc = require('internal/smalloc'); +const deprecate = require('util').deprecate; -exports.alloc = alloc; -exports.copyOnto = copyOnto; -exports.dispose = dispose; -exports.hasExternalData = hasExternalData; +exports.alloc = + deprecate(smalloc.alloc, 'smalloc.alloc: Deprecated, use typed arrays'); + +exports.copyOnto = + deprecate(smalloc.copyOnto, + 'smalloc.copyOnto: Deprecated, use typed arrays'); + +exports.dispose = + deprecate(smalloc.dispose, + 'smalloc.dispose: Deprecated, use typed arrays'); + +exports.hasExternalData = + deprecate(smalloc.hasExternalData, + 'smalloc.hasExternalData: Deprecated, use typed arrays'); -// don't allow kMaxLength to accidentally be overwritten. it's a lot less -// apparent when a primitive is accidentally changed. Object.defineProperty(exports, 'kMaxLength', { - enumerable: true, value: kMaxLength, writable: false + enumerable: true, value: smalloc.kMaxLength, writable: false }); Object.defineProperty(exports, 'Types', { - enumerable: true, value: Object.freeze(smalloc.types), writable: false + enumerable: true, value: Object.freeze(smalloc.Types), writable: false }); - - -// usage: obj = alloc(n[, obj][, type]); -function alloc(n, obj, type) { - n = n >>> 0; - - if (obj === undefined) - obj = {}; - - if (typeof obj === 'number') { - type = obj >>> 0; - obj = {}; - } else if (util.isPrimitive(obj)) { - throw new TypeError('obj must be an Object'); - } - - if (Array.isArray(obj)) - throw new TypeError('obj cannot be an array'); - if (obj instanceof Buffer) - throw new TypeError('obj cannot be a Buffer'); - if (smalloc.isTypedArray(obj)) - throw new TypeError('obj cannot be a typed array'); - if (smalloc.hasExternalData(obj)) - throw new TypeError('object already has external array data'); - - if (type < kMinType || type > kMaxType) - throw new TypeError('unknown external array type: ' + type); - if (n > kMaxLength) - throw new RangeError('Attempt to allocate array larger than maximum ' + - 'size: 0x' + kMaxLength.toString(16) + ' elements'); - - return smalloc.alloc(obj, n, type); -} - - -function dispose(obj) { - if (util.isPrimitive(obj)) - throw new TypeError('obj must be an Object'); - if (obj instanceof Buffer) - throw new TypeError('obj cannot be a Buffer'); - if (smalloc.isTypedArray(obj)) - throw new TypeError('obj cannot be a typed array'); - if (!smalloc.hasExternalData(obj)) - throw new TypeError('obj has no external array data'); - - smalloc.dispose(obj); -} - - -function copyOnto(source, sourceStart, dest, destStart, copyLength) { - if (util.isPrimitive(source)) - throw new TypeError('source must be an Object'); - if (util.isPrimitive(dest)) - throw new TypeError('dest must be an Object'); - if (!smalloc.hasExternalData(source)) - throw new TypeError('source has no external array data'); - if (!smalloc.hasExternalData(dest)) - throw new TypeError('dest has no external array data'); - - return smalloc.copyOnto(source, sourceStart, dest, destStart, copyLength); -} - - -function hasExternalData(obj) { - if (util.isPrimitive(obj)) - return false; - - return smalloc.hasExternalData(obj); -} diff --git a/lib/v8.js b/lib/v8.js index c6cf015..b0b2860 100644 --- a/lib/v8.js +++ b/lib/v8.js @@ -15,7 +15,7 @@ 'use strict'; const v8binding = process.binding('v8'); -const smalloc = require('smalloc'); +const smalloc = require('internal/smalloc'); const heapStatisticsBuffer = smalloc.alloc(v8binding.kHeapStatisticsBufferLength, diff --git a/node.gyp b/node.gyp index 5a7d1ff..847f873 100644 --- a/node.gyp +++ b/node.gyp @@ -71,6 +71,7 @@ 'lib/zlib.js', 'lib/internal/freelist.js', + 'lib/internal/smalloc.js', ], },