[libc-commits] [libc] 77d9591 - Revert "Fix wcpncpy() return value; add test." (#146752)

[via libc-commits]

Author: Michael Jones
Date: 2025-07-02T11:08:21-07:00
New Revision: 77d95911a3a4dc7445280cd5ee217e2a47a41b47

URL: https://github.com/llvm/llvm-project/commit/77d95911a3a4dc7445280cd5ee217e2a47a41b47
DIFF: https://github.com/llvm/llvm-project/commit/77d95911a3a4dc7445280cd5ee217e2a47a41b47.diff

LOG: Revert "Fix wcpncpy() return value; add test." (#146752)

This reverts commit 988876cdd918d5b945a072a3d643b5fa22ae2917.
Was intended to be a PR

Added: 
    

Modified: 
    libc/src/wchar/wcpncpy.cpp
    libc/test/src/wchar/wcpncpy_test.cpp

Removed: 
    


################################################################################
diff  --git a/libc/src/wchar/wcpncpy.cpp b/libc/src/wchar/wcpncpy.cpp
index 0e729db7abf60..9f451b73f07cb 100644
--- a/libc/src/wchar/wcpncpy.cpp
+++ b/libc/src/wchar/wcpncpy.cpp
@@ -28,9 +28,8 @@ LLVM_LIBC_FUNCTION(wchar_t *, wcpncpy,
   for (i = 0; i < n && s2[i] != '\0'; ++i)
     s1[i] = s2[i];
   // When n>strlen(src), n-strlen(src) \0 are appended.
-  for (size_t j = i; j < n; ++j)
-    s1[j] = L'\0';
-  // ...but our result points to the first \0 (if any).
+  for (; i < n; ++i)
+    s1[i] = L'\0';
   return s1 + i;
 }
 

diff  --git a/libc/test/src/wchar/wcpncpy_test.cpp b/libc/test/src/wchar/wcpncpy_test.cpp
index 5ce3e8ce7cd93..98738e230e32d 100644
--- a/libc/test/src/wchar/wcpncpy_test.cpp
+++ b/libc/test/src/wchar/wcpncpy_test.cpp
@@ -75,15 +75,6 @@ TEST(LlvmLibcWCPNCpyTest, CopyTwoWithNull) {
   ASSERT_EQ(dest + 2, res);
 }
 
-TEST(LlvmLibcWCPNCpyTest, CopyAndFill) {
-  wchar_t dest[] = {L'a', L'b', L'c'};
-  wchar_t *res = LIBC_NAMESPACE::wcpncpy(dest, L"x", 3);
-  ASSERT_TRUE(dest[0] == L'x');
-  ASSERT_TRUE(dest[1] == L'\0');
-  ASSERT_TRUE(dest[2] == L'\0');
-  ASSERT_EQ(dest + 1, res);
-}
-
 #if defined(LIBC_ADD_NULL_CHECKS) && !defined(LIBC_HAS_SANITIZER)
 TEST(LlvmLibcWCPNCpyTest, NullptrCrash) {
   // Passing in a nullptr should crash the program.