WebSVN – HelenOS – Diff – /trunk/kernel/generic/src/printf/printf_core.c

     /* Print leading spaces */
     size_t size = strlen_utf8(str);
     if (precision == 0)
         precision = size;
     count_t counter = 0;
     width -= precision;
     if (!(flags & __PRINTF_FLAG_LEFTALIGNED)) {
         while (width-- > 0) {
             if (printf_putchar(' ', ps) == 1)
                 counter++;
+        }
+    }
     int retval;
     size_t bytes = utf8_count_bytes(str, min(size, precision));
     if ((retval = printf_putnchars_utf8(str, bytes, ps)) < 0)
         return -counter;
     while (width-- > 0) {
         if (printf_putchar(' ', ps) == 1)
             counter++;
+    }
     return ((int) counter);
+}
 /** Print UTF-32 string.
+ *
+ *
  */
 int printf_core(const char *fmt, printf_spec_t *ps, va_list ap)
+{
     index_t i = 0;  /* Index of the currently processed character from fmt */
+    index_t nxt = 0;
     index_t j = 0;  /* Index to the first not printed nonformating character */
     wchar_t uc;           /* Current UTF-32 character decoded from fmt */
     count_t counter = 0;  /* Number of UTF-8 characters printed */
     int retval;           /* Return values from nested functions */
+    while (true) {
+        i = nxt;
-    while ((uc = utf8_decode(fmt, &i, UTF8_NO_LIMIT)) != 0) {
+        uc = utf8_decode(fmt, &nxt, UTF8_NO_LIMIT);
+        if (uc == '\0') break;
         /* Control character */
         if (uc == '%') {
             /* Print common characters if any processed */
             if (i > j) {
                 if ((retval = printf_putnchars_utf8(&fmt[j], i - j, ps)) < 0) {
             /* Parse modifiers */
             uint32_t flags = 0;
             bool end = false;
             do {
-                i++;
+                i = nxt;
-                uc = utf8_decode(fmt, &i, UTF8_NO_LIMIT);
+                uc = utf8_decode(fmt, &nxt, UTF8_NO_LIMIT);
                 switch (uc) {
                 case '#':
                     flags |= __PRINTF_FLAG_PREFIX;
                     break;
                 case '-':
             } while (!end);
             /* Width & '*' operator */
             int width = 0;
             if (isdigit(uc)) {
-                while ((uc = utf8_decode(fmt, &i, UTF8_NO_LIMIT)) != 0) {
-                    if (!isdigit(uc))
+                while (true) {
-                        break;
                     width *= 10;
                     width += uc - '0';
-                    i++;
+                    i = nxt;
+                    uc = utf8_decode(fmt, &nxt, UTF8_NO_LIMIT);
+                    if (uc == '\0')
+                        break;
+                    if (!isdigit(uc))
+                        break;
+                }
             } else if (uc == '*') {
                 /* Get width value from argument list */
-                i++;
+                i = nxt;
-                uc = utf8_decode(fmt, &i, UTF8_NO_LIMIT);
+                uc = utf8_decode(fmt, &nxt, UTF8_NO_LIMIT);
                 width = (int) va_arg(ap, int);
                 if (width < 0) {
                     /* Negative width sets '-' flag */
                     width *= -1;
                     flags |= __PRINTF_FLAG_LEFTALIGNED;
+            }
             /* Precision and '*' operator */
             int precision = 0;
             if (uc == '.') {
-                i++;
+                i = nxt;
-                uc = utf8_decode(fmt, &i, UTF8_NO_LIMIT);
+                uc = utf8_decode(fmt, &nxt, UTF8_NO_LIMIT);
                 if (isdigit(uc)) {
-                    while ((uc = utf8_decode(fmt, &i, UTF8_NO_LIMIT)) != 0) {
-                        if (!isdigit(uc))
+                    while (true) {
-                            break;
                         precision *= 10;
                         precision += uc - '0';
-                        i++;
+                        i = nxt;
+                        uc = utf8_decode(fmt, &nxt, UTF8_NO_LIMIT);
+                        if (uc == '\0')
+                            break;
+                        if (!isdigit(uc))
+                            break;
+                    }
-                } else if (fmt[i] == '*') {
+                } else if (uc == '*') {
                     /* Get precision value from the argument list */
-                    i++;
+                    i = nxt;
-                    uc = utf8_decode(fmt, &i, UTF8_NO_LIMIT);
+                    uc = utf8_decode(fmt, &nxt, UTF8_NO_LIMIT);
                     precision = (int) va_arg(ap, int);
                     if (precision < 0) {
                         /* Ignore negative precision */
                         precision = 0;
+                    }
              *        t ptrdiff_t - ISO C 99
              */
             case 'h':
                 /* Char or short */
                 qualifier = PrintfQualifierShort;
-                i++;
+                i = nxt;
-                uc = utf8_decode(fmt, &i, UTF8_NO_LIMIT);
+                uc = utf8_decode(fmt, &nxt, UTF8_NO_LIMIT);
                 if (uc == 'h') {
-                    i++;
+                    i = nxt;
-                    uc = utf8_decode(fmt, &i, UTF8_NO_LIMIT);
+                    uc = utf8_decode(fmt, &nxt, UTF8_NO_LIMIT);
                     qualifier = PrintfQualifierByte;
+                }
                 break;
             case 'l':
                 /* Long or long long */
                 qualifier = PrintfQualifierLong;
-                i++;
+                i = nxt;
-                uc = utf8_decode(fmt, &i, UTF8_NO_LIMIT);
+                uc = utf8_decode(fmt, &nxt, UTF8_NO_LIMIT);
                 if (uc == 'l') {
-                    i++;
+                    i = nxt;
-                    uc = utf8_decode(fmt, &i, UTF8_NO_LIMIT);
+                    uc = utf8_decode(fmt, &nxt, UTF8_NO_LIMIT);
                     qualifier = PrintfQualifierLongLong;
+                }
                 break;
             default:
                 /* Default type */
                     counter = -counter;
                     goto out;
+                }
                 counter += retval;
-                j = i + 1;
+                j = nxt;
                 goto next_char;
             case 'c':
                 if (qualifier == PrintfQualifierLong)
                     retval = print_wchar(va_arg(ap, wchar_t), width, flags, ps);
                 else
                     counter = -counter;
                     goto out;
                 };
                 counter += retval;
-                j = i + 1;
+                j = nxt;
                 goto next_char;
             /*
              * Integer values
              */
                 counter = -counter;
                 goto out;
+            }
             counter += retval;
-            j = i + 1;
+            j = nxt;
+        }
 next_char:
+        ;
-        i++;
+    }
     if (i > j) {
         if ((retval = printf_putnchars_utf8(&fmt[j], i - j, ps)) < 0) {
             /* Error */

Subversion Repositories HelenOS

(root)/trunk/kernel/generic/src/printf/printf_core.c – Rev 4185 → 4199