Revert "Introduce function IsNullTerminated"

This reverts commit 6827079.

Author: mjklemm

flang-rt/include/flang-rt/runtime/tools.h

@@ -525,9 +525,6 @@ RT_API_ATTRS void ShallowCopy(const Descriptor &to, const Descriptor &from,
     bool toIsContiguous, bool fromIsContiguous);
 RT_API_ATTRS void ShallowCopy(const Descriptor &to, const Descriptor &from);
 
-// Determines if a character string is null-terminated.
-RT_API_ATTRS bool IsNullTerminated(char *str, std::size_t length);
-
 // Ensures that a character string is null-terminated, allocating a /p length +1
 // size memory for null-terminator if necessary. Returns the original or a newly
 // allocated null-terminated string (responsibility for deallocation is on the

flang-rt/lib/runtime/tools.cpp

@@ -273,13 +273,9 @@ RT_API_ATTRS void ShallowCopy(const Descriptor &to, const Descriptor &from) {
   ShallowCopy(to, from, to.IsContiguous(), from.IsContiguous());
 }
 
-RT_API_ATTRS bool IsNullTerminated(char *str, std::size_t length) {
-  return !(runtime::memchr(str, '\0', length) == nullptr);
-}
-
 RT_API_ATTRS char *EnsureNullTerminated(
     char *str, std::size_t length, Terminator &terminator) {
-  if (!IsNullTerminated(str, length)) {
+  if (runtime::memchr(str, '\0', length) == nullptr) {
     char *newCmd{(char *)AllocateMemoryOrCrash(terminator, length + 1)};
     runtime::memcpy(newCmd, str, length);
     newCmd[length] = '\0';