You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@subversion.apache.org by br...@apache.org on 2018/11/25 05:38:02 UTC

svn commit: r1847385 - /subversion/trunk/subversion/svn/list-cmd.c

Author: brane
Date: Sun Nov 25 05:38:02 2018
New Revision: 1847385

URL: http://svn.apache.org/viewvc?rev=1847385&view=rev
Log:
* subversion/svn/list-cmd.c (get_human_readable_size):
   Fix a comment to avoid confusing wishful thinking and reality.

Modified:
    subversion/trunk/subversion/svn/list-cmd.c

Modified: subversion/trunk/subversion/svn/list-cmd.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/svn/list-cmd.c?rev=1847385&r1=1847384&r2=1847385&view=diff
==============================================================================
--- subversion/trunk/subversion/svn/list-cmd.c (original)
+++ subversion/trunk/subversion/svn/list-cmd.c Sun Nov 25 05:38:02 2018
@@ -115,7 +115,8 @@ get_human_readable_size(svn_filesize_t s
                          : (size >> 3 * index) / 128.0 / index);
 
   /* When the absolute adjusted size is < 10, show tenths of a unit, too.
-     NOTE: This will display the locale-specific decimal separator. */
+     NOTE: This *should* display the locale-specific decimal separator,
+           but apparently APR isn't too hot on localisation. */
   return apr_psprintf(scratch_pool, "%.*f%c",
                       (abs_size >> 10 * index < 10) ? 1 : 0,
                       human_readable_size, order[index].suffix);