[PATCH 03/10] lib: implement simple_strtoll
Sam Ravnborg
sam at ravnborg.org
Mon Apr 10 05:44:48 PDT 2017
Hi Sascha.
Nitpick mode...
> --- a/lib/strtox.c
> +++ b/lib/strtox.c
> @@ -65,3 +65,11 @@ unsigned long long simple_strtoull (const char *cp, char **endp, unsigned int ba
> return result;
> }
> EXPORT_SYMBOL(simple_strtoull);
> +
> +long long simple_strtoll(const char *cp,char **endp,unsigned int base)
> +{
> + if(*cp=='-')
It hurts my eyes to see the missing space between if and "(".
simple_strtol() suffer from the same issue, but in the rest of the file
there is a space between if and "("
And on top I did not see simple_strtoull() used in the patch series.
But then I may have missed it as I just skimmed the patches.
Everything else I saw looked good.
Sam
More information about the barebox
mailing list