Commit d223e3c
committed
src: make
`AsyncResource` is intended to be a base class, and since we don’t
know what API consumers will do with it in their own code,
it’s good practice to make its destructor virtual.
This should not be ABI-breaking since all class methods are inline.
PR-URL: #20633
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>AsyncResource destructor virtual1 parent b795953 commit d223e3cCopy full SHA for d223e3c
File tree
Expand file treeCollapse file tree
3 files changed
+25
-1
lines changedOpen diff view settings
Filter options
- src
- test/addons/async-resource
Expand file treeCollapse file tree
3 files changed
+25
-1
lines changedOpen diff view settings
Collapse file
+1-1Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
712 | 712 | |
713 | 713 | |
714 | 714 | |
715 | | - |
| 715 | + |
716 | 716 | |
717 | 717 | |
718 | 718 | |
|
Collapse file
test/addons/async-resource/binding.cc
Copy file name to clipboardExpand all lines: test/addons/async-resource/binding.cc+22Lines changed: 22 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
17 | 17 | |
18 | 18 | |
19 | 19 | |
| 20 | + |
| 21 | + |
| 22 | + |
| 23 | + |
| 24 | + |
| 25 | + |
| 26 | + |
| 27 | + |
| 28 | + |
| 29 | + |
| 30 | + |
20 | 31 | |
21 | 32 | |
22 | 33 | |
| ||
98 | 109 | |
99 | 110 | |
100 | 111 | |
| 112 | + |
| 113 | + |
| 114 | + |
| 115 | + |
| 116 | + |
| 117 | + |
| 118 | + |
| 119 | + |
| 120 | + |
| 121 | + |
101 | 122 | |
102 | 123 | |
103 | 124 | |
| ||
107 | 128 | |
108 | 129 | |
109 | 130 | |
| 131 | + |
110 | 132 | |
111 | 133 | |
112 | 134 | |
|
Collapse file
test/addons/async-resource/test.js
Copy file name to clipboardExpand all lines: test/addons/async-resource/test.js+2Lines changed: 2 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
5 | 5 | |
6 | 6 | |
7 | 7 | |
| 8 | + |
| 9 | + |
8 | 10 | |
9 | 11 | |
10 | 12 | |
|
0 commit comments