[PATCH 03/10] lib: implement simple_strtoll

Sascha Hauer s.hauer at pengutronix.de
Mon Apr 10 23:42:28 PDT 2017


On Mon, Apr 10, 2017 at 02:44:48PM +0200, Sam Ravnborg wrote:
> 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 "(".

You're right. The whitespace placing throughout the file doesn't help
reading it. I just sent a patch cleaning it up and also fixed the
whitespaces in this patch.

> 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.

I'm pretty sure it's used ;)

Sascha


-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |



More information about the barebox mailing list