Author: rinrab
Date: Wed May 20 09:00:30 2026
New Revision: 1934427

Log:
Follow-up to r1934408: Adjust outdated comment.

* subversion/libsvn_subr/utf8proc.c
  (svn_utf_cstring_utf8_width): We don't call svn_utf__cstring_is_valid()
   anymore so it actually needs error checking. Lucky for us, utf8proc_iterate()
   does all that. That's really the reason why we removed validation in the
   first place.

Suggested by: kotkov

Modified:
   subversion/trunk/subversion/libsvn_subr/utf8proc.c

Modified: subversion/trunk/subversion/libsvn_subr/utf8proc.c
==============================================================================
--- subversion/trunk/subversion/libsvn_subr/utf8proc.c  Wed May 20 08:55:33 
2026        (r1934426)
+++ subversion/trunk/subversion/libsvn_subr/utf8proc.c  Wed May 20 09:00:30 
2026        (r1934427)
@@ -616,7 +616,7 @@ svn_utf_cstring_utf8_width(const char *c
 
   /* Convert the UTF-8 string to UTF-32 (UCS4) which is the format
    * utf8proc_charwidth() expects, and get the width of each character.
-   * We don't need much error checking since the input is valid UTF-8. */
+   * utf8proc_iterate() does all the error checking we might ever need. */
   while (*cstr)
     {
       apr_int32_t ucs;

Reply via email to