benchmark: replace hasOwnProperty() with Object.hasOwn() · nodejs/node@ca1cd42 · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 633
• Pull requests 759
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit ca1cd42

Browse filesBrowse the repository at this point in the historyBrowse files

Trott

authored and

ruyadorno

committed

benchmark: replace hasOwnProperty() with Object.hasOwn()

PR-URL: #41724 Reviewed-By: Tierney Cyren <hello@bnb.im> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Mestery <mestery@protonmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net>

1 parent f5adf54 commit ca1cd42
Copy full SHA for ca1cd42

1 file changed

+1-1Lines changed: 1 addition & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• benchmark
  • common.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎benchmark/common.js‎

Copy file name to clipboardExpand all lines: benchmark/common.js

+1-1Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -38,7 +38,7 @@ class Benchmark {
38	38	this.config = this.queue[0];
39	39
40	40	process.nextTick(() => {
41		- if (process.env.hasOwnProperty('NODE_RUN_BENCHMARK_FN')) {
	41	+ if (Object.hasOwn(process.env, 'NODE_RUN_BENCHMARK_FN')) {
42	42	fn(this.config);
43	43	} else {
44	44	// _run will use fork() to create a new process for each configuration

0 commit comments

Comments

0 (0)