From 1fcb76e8f2f05fc084337da097a17b06cc7d0c68 Mon Sep 17 00:00:00 2001
From: James M Snell <jasnell@gmail.com>
Date: Thu, 15 Jun 2017 08:31:11 -0700
Subject: [PATCH] cluster: remove deprecated property
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

PR-URL: https://github.com/nodejs/node/pull/13702
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
---
 doc/api/cluster.md             | 34 ----------------------------------
 lib/internal/cluster/worker.js | 14 --------------
 2 files changed, 48 deletions(-)

diff --git a/doc/api/cluster.md b/doc/api/cluster.md
index c7d8359ac0ce93..721b33a7e184f1 100644
--- a/doc/api/cluster.md
+++ b/doc/api/cluster.md
@@ -451,40 +451,6 @@ if (cluster.isMaster) {
 }
 ```
 
-### worker.suicide
-<!-- YAML
-added: v0.7.0
-deprecated: v6.0.0
-changes:
-  - version: v7.0.0
-    pr-url: https://github.com/nodejs/node/pull/3747
-    description: Accessing this property will now emit a deprecation warning.
--->
-
-> Stability: 0 - Deprecated: Use [`worker.exitedAfterDisconnect`][] instead.
-
-An alias to [`worker.exitedAfterDisconnect`][].
-
-Set by calling `.kill()` or `.disconnect()`. Until then, it is `undefined`.
-
-The boolean `worker.suicide` is used to distinguish between voluntary
-and accidental exit, the master may choose not to respawn a worker based on
-this value.
-
-```js
-cluster.on('exit', (worker, code, signal) => {
-  if (worker.suicide === true) {
-    console.log('Oh, it was just voluntary – no need to worry');
-  }
-});
-
-// kill worker
-worker.kill();
-```
-
-This API only exists for backwards compatibility and will be removed in the
-future.
-
 ## Event: 'disconnect'
 <!-- YAML
 added: v0.7.9
diff --git a/lib/internal/cluster/worker.js b/lib/internal/cluster/worker.js
index 687e4c12fecea4..2cf5fc385809c7 100644
--- a/lib/internal/cluster/worker.js
+++ b/lib/internal/cluster/worker.js
@@ -1,10 +1,6 @@
 'use strict';
 const EventEmitter = require('events');
-const internalUtil = require('internal/util');
 const util = require('util');
-const defineProperty = Object.defineProperty;
-const suicideDeprecationMessage =
-    'worker.suicide is deprecated. Please use worker.exitedAfterDisconnect.';
 
 module.exports = Worker;
 
@@ -20,16 +16,6 @@ function Worker(options) {
 
   this.exitedAfterDisconnect = undefined;
 
-  defineProperty(this, 'suicide', {
-    get: internalUtil.deprecate(
-      () => this.exitedAfterDisconnect,
-      suicideDeprecationMessage, 'DEP0007'),
-    set: internalUtil.deprecate(
-      (val) => { this.exitedAfterDisconnect = val; },
-      suicideDeprecationMessage, 'DEP0007'),
-    enumerable: true
-  });
-
   this.state = options.state || 'none';
   this.id = options.id | 0;