svn commit: r1934427 - subversion/trunk/subversion/libsvn_subr

[email protected]
Newsgroups gmane.comp.version-control.subversion.svn
Message-ID <177926763104.1159112.7583346830402058759@svn03-he-fi>
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;
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.