doc,worker: deprecate --trace-atomics-wait

kvakil · juanarbol · commit 8f6bd5a40203 · 2022-10-11T14:45:10.000-05:00
V8 has asked if it possible to remove the functionality underlying `--trace-atomics-wait`. Let's start with a documentation-only deprecation. PR-URL: #44093 Refs: #42982 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Feng Yu <F3n67u@outlook.com>
diff --git a/doc/api/cli.md b/doc/api/cli.md
@@ -1195,8 +1195,11 @@ for TLSv1.2, which is not as secure as TLSv1.3.
 
 <!-- YAML
 added: v14.3.0
+deprecated: REPLACEME
 -->
 
+> Stability: 0 - Deprecated
+
 Print short summaries of calls to [`Atomics.wait()`][] to stderr.
 The output could look like this:
 
diff --git a/doc/api/deprecations.md b/doc/api/deprecations.md
@@ -3134,13 +3134,27 @@ Type: Documentation-only
 `code` values other than `undefined`, `null`, integer numbers and integer
 strings (e.g., '1') are deprecated as parameter in [`process.exit()`][].
 
+### DEP0165: `--trace-atomics-wait`
+
+<!-- YAML
+changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/44093
+    description: Documentation-only deprecation.
+-->
+
+Type: Documentation-only
+
+The [`--trace-atomics-wait`][] flag is deprecated.
+
 [Legacy URL API]: url.md#legacy-url-api
 [NIST SP 800-38D]: https://nvlpubs.nist.gov/nistpubs/Legacy/SP/nistspecialpublication800-38d.pdf
 [RFC 6066]: https://tools.ietf.org/html/rfc6066#section-3
 [WHATWG URL API]: url.md#the-whatwg-url-api
 [`"exports"` or `"main"` entry]: packages.md#main-entry-point-export
 [`--pending-deprecation`]: cli.md#--pending-deprecation
 [`--throw-deprecation`]: cli.md#--throw-deprecation
+[`--trace-atomics-wait`]: cli.md#--trace-atomics-wait
 [`--unhandled-rejections`]: cli.md#--unhandled-rejectionsmode
 [`Buffer.allocUnsafeSlow(size)`]: buffer.md#static-method-bufferallocunsafeslowsize
 [`Buffer.from(array)`]: buffer.md#static-method-bufferfromarray
diff --git a/doc/node.1 b/doc/node.1
@@ -429,6 +429,7 @@ favour of TLSv1.3, which is more secure.
 Print short summaries of calls to
 .Sy Atomics.wait() .
 .
+This flag is deprecated.
 .It Fl -trace-deprecation
 Print stack traces for deprecations.
 .
diff --git a/src/node_options.cc b/src/node_options.cc
@@ -539,7 +539,7 @@ EnvironmentOptionsParser::EnvironmentOptionsParser() {
             &EnvironmentOptions::throw_deprecation,
             kAllowedInEnvironment);
   AddOption("--trace-atomics-wait",
-            "trace Atomics.wait() operations",
+            "(deprecated) trace Atomics.wait() operations",
             &EnvironmentOptions::trace_atomics_wait,
             kAllowedInEnvironment);
   AddOption("--trace-deprecation",

Original file line number	Diff line number	Diff line change
`@@ -429,6 +429,7 @@ favour of TLSv1.3, which is more secure.`
`429`	`429`	`Print short summaries of calls to`
`430`	`430`	`.Sy Atomics.wait() .`
`431`	`431`	`.`
	`432`	`+This flag is deprecated.`
`432`	`433`	`.It Fl -trace-deprecation`
`433`	`434`	`Print stack traces for deprecations.`
`434`	`435`	`.`