developit/workerize

Consider removing expose(), call() and kill() ?

Open

#26 创建于 2018年3月20日

在 GitHub 查看
 (4 评论) (0 反应) (0 负责人)JavaScript (4,299 star) (113 fork)batch import
enhancementhelp wanted

描述

	worker.expose = methodName => {
		worker[i] = function() {
			return worker.call(methodName, [].slice.call(arguments));
		};
	};

Instead methodName parameter should be used:

	worker.expose = methodName => {
		worker[methodName] = function() {
			return worker.call(methodName, [].slice.call(arguments));
		};
	};

Beside that I don't see why the call and expose methods are accessible from outside. (Don't assign to worker at all) The purpose of the kill method is also not clear for me.

贡献者指南