[libc-commits] [libc] [libc] s/NULL/nullptr (PR #86867)
Nick Desaulniers via libc-commits
libc-commits at lists.llvm.org
Wed Mar 27 13:27:12 PDT 2024
https://github.com/nickdesaulniers created https://github.com/llvm/llvm-project/pull/86867
Otherwise we need to pull in stddef.h for the declaration of NULL.
>From 6bb12a552ad5ee2823351fa2a03cb5df54a1c48e Mon Sep 17 00:00:00 2001
From: Nick Desaulniers <ndesaulniers at google.com>
Date: Wed, 27 Mar 2024 13:26:28 -0700
Subject: [PATCH] [libc] s/NULL/nullptr
Otherwise we need to pull in stddef.h for the declaration of NULL.
---
libc/src/__support/char_vector.h | 8 ++++----
libc/src/stdlib/str_from_util.h | 2 +-
libc/src/stdlib/strtod.cpp | 2 +-
libc/src/stdlib/strtof.cpp | 2 +-
libc/src/stdlib/strtold.cpp | 2 +-
5 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/libc/src/__support/char_vector.h b/libc/src/__support/char_vector.h
index 955abdc1fa5ae0..d39310e09dd71e 100644
--- a/libc/src/__support/char_vector.h
+++ b/libc/src/__support/char_vector.h
@@ -11,8 +11,8 @@
#include "src/__support/common.h" // LIBC_INLINE
-#include <stddef.h>
-#include <stdlib.h> // For allocation.
+#include <stddef.h> // size_t
+#include <stdlib.h> // malloc, realloc, free
namespace LIBC_NAMESPACE {
@@ -46,7 +46,7 @@ class CharVector {
if (cur_str == local_buffer) {
char *new_str;
new_str = reinterpret_cast<char *>(malloc(cur_buff_size));
- if (new_str == NULL) {
+ if (new_str == nullptr) {
return false;
}
// TODO: replace with inline memcpy
@@ -55,7 +55,7 @@ class CharVector {
cur_str = new_str;
} else {
cur_str = reinterpret_cast<char *>(realloc(cur_str, cur_buff_size));
- if (cur_str == NULL) {
+ if (cur_str == nullptr) {
return false;
}
}
diff --git a/libc/src/stdlib/str_from_util.h b/libc/src/stdlib/str_from_util.h
index c4c1c0a0ba4ef0..58afa98afc08ed 100644
--- a/libc/src/stdlib/str_from_util.h
+++ b/libc/src/stdlib/str_from_util.h
@@ -11,7 +11,7 @@
// %{a,A,e,E,f,F,g,G}, are not allowed and any code that does otherwise results
// in undefined behaviour(including use of a '%%' conversion specifier); which
// in this case is that the buffer string is simply populated with the format
-// string. The case of the input being NULL should be handled in the calling
+// string. The case of the input being nullptr should be handled in the calling
// function (strfromf, strfromd, strfroml) itself.
#ifndef LLVM_LIBC_SRC_STDLIB_STRFROM_UTIL_H
diff --git a/libc/src/stdlib/strtod.cpp b/libc/src/stdlib/strtod.cpp
index db5e0edefb5b38..461f7feb5bf63d 100644
--- a/libc/src/stdlib/strtod.cpp
+++ b/libc/src/stdlib/strtod.cpp
@@ -19,7 +19,7 @@ LLVM_LIBC_FUNCTION(double, strtod,
if (result.has_error())
libc_errno = result.error;
- if (str_end != NULL)
+ if (str_end != nullptr)
*str_end = const_cast<char *>(str + result.parsed_len);
return result.value;
diff --git a/libc/src/stdlib/strtof.cpp b/libc/src/stdlib/strtof.cpp
index 2cc8829f63d34c..554d096879c593 100644
--- a/libc/src/stdlib/strtof.cpp
+++ b/libc/src/stdlib/strtof.cpp
@@ -19,7 +19,7 @@ LLVM_LIBC_FUNCTION(float, strtof,
if (result.has_error())
libc_errno = result.error;
- if (str_end != NULL)
+ if (str_end != nullptr)
*str_end = const_cast<char *>(str + result.parsed_len);
return result.value;
diff --git a/libc/src/stdlib/strtold.cpp b/libc/src/stdlib/strtold.cpp
index 7378963f21b240..9c3e1db900670a 100644
--- a/libc/src/stdlib/strtold.cpp
+++ b/libc/src/stdlib/strtold.cpp
@@ -19,7 +19,7 @@ LLVM_LIBC_FUNCTION(long double, strtold,
if (result.has_error())
libc_errno = result.error;
- if (str_end != NULL)
+ if (str_end != nullptr)
*str_end = const_cast<char *>(str + result.parsed_len);
return result.value;
More information about the libc-commits
mailing list