On 4/30/19 9:52 AM, Alex Bennée wrote: > @@ -119,6 +119,10 @@ void ml_printf(const char *fmt, ...) > str = va_arg(ap, char*); > print_str(str); > break; > + case 'c': > + c = (char) va_arg(ap, int); > + __sys_outc(c); > + break;
The assignment and cast are both unnecessary.
You could just as well write
__sys_outc(va_arg(ap, int))
but it's not wrong so
Reviewed-by: Richard Henderson <[email protected]>
r~
