Skip to content

Commit a416692

Browse files
committed
repl: remove deprecated repl.memory function
This removes the memory function. It is deprecated for a long time while not being really helpful being exposed. Thus, it is removed to improve maintainability of the REPL module. Signed-off-by: Ruben Bridgewater <[email protected]> PR-URL: nodejs#33286 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent f217b2d commit a416692

File tree

3 files changed

+4
-21
lines changed

3 files changed

+4
-21
lines changed

doc/api/deprecations.md

+4-1
Original file line numberDiff line numberDiff line change
@@ -1696,12 +1696,15 @@ file descriptors.
16961696
### DEP0082: `REPLServer.prototype.memory()`
16971697
<!-- YAML
16981698
changes:
1699+
- version: REPLACEME
1700+
pr-url: https://github.com/nodejs/node/pull/33286
1701+
description: End-of-Life.
16991702
- version: v9.0.0
17001703
pr-url: https://github.com/nodejs/node/pull/16242
17011704
description: Runtime deprecation.
17021705
-->
17031706
1704-
Type: Runtime
1707+
Type: End-of-Life
17051708
17061709
`REPLServer.prototype.memory()` is only necessary for the internal mechanics of
17071710
the `REPLServer` itself. Do not use this function.

lib/repl.js

-5
Original file line numberDiff line numberDiff line change
@@ -1352,11 +1352,6 @@ REPLServer.prototype.defineCommand = function(keyword, cmd) {
13521352
this.commands[keyword] = cmd;
13531353
};
13541354

1355-
REPLServer.prototype.memory = deprecate(
1356-
_memory,
1357-
'REPLServer.memory() is deprecated',
1358-
'DEP0082');
1359-
13601355
// TODO(BridgeAR): This should be replaced with acorn to build an AST. The
13611356
// language became more complex and using a simple approach like this is not
13621357
// sufficient anymore.

test/parallel/test-repl-memory-deprecation.js

-15
This file was deleted.

0 commit comments

Comments
 (0)