Not even worth it...

[jimmywarting]

u8array/index.js

Lines 1 to 18 in 178cb30

	const u8 = (src) => {
	let string = '';
	if (typeof src !== 'string') {
	string = src.toString();
	} else {
	string = src;
	}
	const arrayBuffer = new ArrayBuffer(string.length * 1);
	const newUint = new Uint8Array(arrayBuffer);
	newUint.forEach((_, i) => {
	newUint[i] = string.charCodeAt(i);
	});
	return newUint;
	};
	module.exports = u8;

Can all be replaced by this:

var e = new TextEncoder()
module.exports = x => e.encode(x)

Would discourage use of this package and instead encourage ppl to use TextEncoder instead

[nothingrandom]

I do agree that the use of this node package is diminishing, I would like to note that the implementation of TextEncoder is still experimental, and requires a polyfill for IE.

Further, this package does a string check, which can be useful for when a number gets passed in instead.

[jimmywarting]

webidl have stuff that makes sure everything you pass to encode() is transformed into a string first by doing more magic than just String(obj)
you can pass in boolean, numbers, null, whatever...

and IE is pretty much dead now... think it would just be better to conditionally load in a pollyfill if TextEncoder isn't available