Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit d8ac55a

Browse filesBrowse files
danbevMylesBorins
authored andcommitted
test: use NULL instead of 0 in common.h
This commit updates the macros in test/addons-napi/common.h to use NULL instead of 0. This is very minor, but I had to look twice while going through the code and finding what the macro was doing and comparing with the struct definition. Using NULL makes it a little clearer I think. PR-URL: #24104 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent 5dfc1bb commit d8ac55a
Copy full SHA for d8ac55a

File tree

Expand file treeCollapse file tree

1 file changed

+2
-2
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

1 file changed

+2
-2
lines changed
Open diff view settings
Collapse file

‎test/addons-napi/common.h‎

Copy file name to clipboardExpand all lines: test/addons-napi/common.h
+2-2Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@
5454
NAPI_CALL_BASE(env, the_call, NAPI_RETVAL_NOTHING)
5555

5656
#define DECLARE_NAPI_PROPERTY(name, func) \
57-
{ (name), 0, (func), 0, 0, 0, napi_default, 0 }
57+
{ (name), NULL, (func), NULL, NULL, NULL, napi_default, NULL }
5858

5959
#define DECLARE_NAPI_GETTER(name, func) \
60-
{ (name), 0, 0, (func), 0, 0, napi_default, 0 }
60+
{ (name), NULL, NULL, (func), NULL, NULL, napi_default, NULL }

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.