Changeset 80496 in vbox for trunk/src/VBox/Runtime/common/string

Timestamp:

Aug 29, 2019 11:58:21 AM (5 years ago)

Author:

vboxsync

Message:

IPRT/string.h: Added added two more human readable size formatting variants for skipping the 'i' in 'KiB' and such. Made the human readable size formatting use the space format flag to specify a space between the value and the unit.

File:

• trunk/src/VBox/Runtime/common/string/strformatrt.cpp (modified) (6 diffs)

trunk/src/VBox/Runtime/common/string/strformatrt.cpp

@@ -989 +989 @@
 #endif /* IN_RING3 */
 
+                    /*
+                     * Human readable sizes.
+                     */
                     case 'c':
                     case 'u':
@@ -997 +1000 @@
                         uint64_t    uFraction = 0;
                         const char *pszPrefix = NULL;
-                        unsigned    cchFixedPart;
                         char        ch2 = *(*ppszFormat)++;
-                        AssertMsgReturn(ch2 == 'b' || ch2 == 'i', ("invalid type '%.10s'!\n", pszFormatOrg), 0);
+                        AssertMsgReturn(ch2 == 'b' || ch2 == 'B' || ch2 == 'i', ("invalid type '%.10s'!\n", pszFormatOrg), 0);
                         uValue = va_arg(*pArgs, uint64_t);
 
                         if (!(fFlags & RTSTR_F_PRECISION))
-                            cchPrecision = 1;
+                            cchPrecision = 1; /** @todo default to flexible decimal point. */
                         else if (cchPrecision > 3)
                             cchPrecision = 3;
@@ -1009 +1011 @@
                             cchPrecision = 0;
 
-                        cchFixedPart = cchPrecision + (cchPrecision != 0) + (ch == 'c');
-
-                        if (ch2 == 'b')
+                        if (ch2 == 'b' || ch2 == 'B')
                         {
                             static const struct
@@ -1040 +1040 @@
                                     uValue  >>= s_aUnits[i].cShift;
                                     pszPrefix = s_aUnits[i].pszPrefix;
-                                    cchFixedPart += 2;
                                     break;
                                 }
@@ -1075 +1074 @@
                                     }
                                     pszPrefix = s_aUnits[i].pszPrefix;
-                                    cchFixedPart += 1;
                                     break;
                                 }
@@ -1089 +1087 @@
                                                          RTSTR_F_ZEROPAD | RTSTR_F_WIDTH);
                             }
+                            if (fFlags & RTSTR_F_BLANK)
+                                szBuf[cchBuf++] = ' ';
                             szBuf[cchBuf++] = *pszPrefix++;
-                            if (*pszPrefix)
+                            if (*pszPrefix && ch2 != 'B')
                                 szBuf[cchBuf++] = *pszPrefix;
                         }
+                        else if (fFlags & RTSTR_F_BLANK)
+                            szBuf[cchBuf++] = ' ';
                         if (ch == 'c')
                             szBuf[cchBuf++] = 'B';