Commit fca56a4
src: add missing <cstdlib> for abort() declaration
builtin_info.cc calls `abort()` but didn't call `<cstdlib>` to pick up
its declaration.
This has been working because `abort()` got picked up by a transitive
declaration from another header, but it broke in Electron when we
rolled our LLVM libc++ snapshot from commit 8e870711923 to 9aaa9a2ee87.
Signed-off-by: Charles Kerr <charles@charleskerr.com>
PR-URL: #63001
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Richard Lau <richard.lau@ibm.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>1 parent 712a15d commit fca56a4
1 file changed
Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1,4 +1,5 @@ | |||
| 1 | 1 | #include "builtin_info.h" | |
| 2 | + #include <cstdlib> | ||
| 2 | 3 | ||
| 3 | 4 | namespace node { | |
| 4 | 5 | namespace builtins { | |
@@ -41,7 +42,7 @@ std::string GetBuiltinSourceTypeName(BuiltinSourceType type) { | |||
| 41 | 42 | case BuiltinSourceType::kSourceTextModule: | |
| 42 | 43 | return "kSourceTextModule"; | |
| 43 | 44 | } | |
| 44 | - abort(); | ||
| 45 | + std::abort(); | ||
| 45 | 46 | } | |
| 46 | 47 | ||
| 47 | 48 | } // namespace builtins | |
0 commit comments