Commit bac4c41
src: deprecate option variables in public API
These variables should never have been exposed as part of the
public API, and certainly not as variables. Using CLI options
parser is the right thing to do here, at least until we expose
some part of the options parser API publicly (which should be
possible to do now).
PR-URL: #22515
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>1 parent 9565029 commit bac4c41
2 files changed
Lines changed: 12 additions & 8 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -199,14 +199,17 @@ typedef intptr_t ssize_t; | |||
| 199 | 199 | ||
| 200 | 200 | namespace node { | |
| 201 | 201 | ||
| 202 | - // TODO(addaleax): Deprecate and remove all of these ASAP. They have been | ||
| 203 | - // made effectively non-functional anyway. | ||
| 204 | - NODE_EXTERN extern bool no_deprecation; | ||
| 202 | + // TODO(addaleax): Remove all of these. | ||
| 203 | + NODE_DEPRECATED("use command-line flags", | ||
| 204 | + NODE_EXTERN extern bool no_deprecation); | ||
| 205 | 205 | #if HAVE_OPENSSL | |
| 206 | - NODE_EXTERN extern bool ssl_openssl_cert_store; | ||
| 206 | + NODE_DEPRECATED("use command-line flags", | ||
| 207 | + NODE_EXTERN extern bool ssl_openssl_cert_store); | ||
| 207 | 208 | # if NODE_FIPS_MODE | |
| 208 | - NODE_EXTERN extern bool enable_fips_crypto; | ||
| 209 | - NODE_EXTERN extern bool force_fips_crypto; | ||
| 209 | + NODE_DEPRECATED("use command-line flags", | ||
| 210 | + NODE_EXTERN extern bool enable_fips_crypto); | ||
| 211 | + NODE_DEPRECATED("user command-line flags", | ||
| 212 | + NODE_EXTERN extern bool force_fips_crypto); | ||
| 210 | 213 | # endif | |
| 211 | 214 | #endif | |
| 212 | 215 | ||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -27,8 +27,9 @@ | |||
| 27 | 27 | ||
| 28 | 28 | namespace node { | |
| 29 | 29 | ||
| 30 | - // TODO(addaleax): Deprecate and remove this ASAP. | ||
| 31 | - extern bool zero_fill_all_buffers; | ||
| 30 | + // TODO(addaleax): Remove this. | ||
| 31 | + NODE_DEPRECATED("use command-line flags", | ||
| 32 | + extern bool zero_fill_all_buffers); | ||
| 32 | 33 | ||
| 33 | 34 | namespace Buffer { | |
| 34 | 35 | ||
0 commit comments