[ros-diffs] [arty] 22680: Fixed actual bug in freeldr printf. Probably used to depend on a bug in _itoa, which was later made compliant. I'm guessing that this went unnoticed because nobody used freeldr's printf very often.

Author: arty
Date: Thu Jun 29 02:26:04 2006
New Revision: 22680

URL: http://svn.reactos.org/svn/reactos?rev=22680&view=rev
Log:
Fixed actual bug in freeldr printf.  Probably used to depend on a bug in 
_itoa, which was later made compliant.  I'm guessing that this went 
unnoticed because nobody used freeldr's printf very often.

Modified:
    branches/powerpc/reactos/boot/freeldr/freeldr/ui/tui.c

Modified: branches/powerpc/reactos/boot/freeldr/freeldr/ui/tui.c
URL:
http://svn.reactos.org/svn/reactos/branches/powerpc/reactos/boot/freeldr/fr…
==============================================================================
--- branches/powerpc/reactos/boot/freeldr/freeldr/ui/tui.c (original)
+++ branches/powerpc/reactos/boot/freeldr/freeldr/ui/tui.c Thu Jun 29 02:26:04 2006
@@ -22,6 +22,8 @@
 PVOID	TextVideoBuffer = NULL;
 extern BOOLEAN UiDrawTime;
 extern BOOLEAN UiMinimal;
+extern void ofw_print_string(const char *);
+extern void ofw_print_number(int x);
 
 /*
  * printf() - prints formatted text to stdout
@@ -43,19 +45,20 @@
 		{
 			switch (c = *(format++))
 			{
-			case 'd': case 'u': case 'x':
-                if (c == 'x')
-                    *_itoa(va_arg(ap, unsigned long), str, 16) = 0;
-                else
-                    *_itoa(va_arg(ap, unsigned long), str, 10) = 0;
+			case 'd': case 'u': case 'x': {
+				unsigned long x = va_arg(ap, unsigned long);
+                		if (c == 'x')
+                		    _itoa(x, str, 16);
+                		else
+                    		    _itoa(x, str, 10);
 
 				ptr = str;
 
 				while (*ptr)
 				{
-					MachConsPutChar(*(ptr++));
+				    MachConsPutChar(*(ptr++));
 				}
-				break;
+			} break;
 
 			case 'c': MachConsPutChar((va_arg(ap,int))&0xff); break;