Re: [PATCH -v1.1] x86, boot: Further compress CPUs bootup message

[Ingo Molnar]

* Borislav Petkov <bp@xxxxxxxxx> wrote:

> @@ -2,6 +2,10 @@ int num_digits(int val)
>  {
>  	int digits = 0;
>  
> +	/* Handle special case */
> +	if (!val)
> +		return 1;
> +
>  	while (val) {
>  		val /= 10;
>  		digits++;

Hm. I suspect this could then be written as:

int num_digits(int val)
{
	int digits = 0;

	do {
		val /= 10;
		digits++;
	} while (val);

	return digits;
}

No ugly special case! :-)

Thanks,

	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/