[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/fre...
==============================================================================
--- 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;