For timeit magic, no s at "loops" if only one loop

[Naereen]

Just a tiny tiny grammatical correction: no s at "loops" if there is only one loop.
It affects the %timeit magic, its doc, and the message about "an intermediate result being cached".

A message like "1 loops, best of 3: 307 ms per loop" always bothered me 😄 as there is only one loop.
So loops → loop if there is only one.

[Carreau]

Thanks that's great !
I suppose we don't have to handle the case where there is 0 loops :-P

[Naereen]

Well I was wondering about it, but it should never arise right?

Even using -n0 parameter does not force 0 loop:

In [22]: %timeit -n0 time.sleep(2)
1 loops, best of 3: 2 s per loop

[Carreau]

Yep, I think the 0 case if fine as is.
I guess we could consider 0 loop if the code was raising an exception, but then we never get to the printing part.

One other way would have been :

u"%d loop%s, best of %d: %s per loop" % ('' if self.loops == 1 else 's' , ...)

It should handle correctly 0, but that's bikeshedding.

[Naereen]

I know the inline .. if .. else .. syntax, but it would have been more confuse here, the ".." % (..) part of the line was already quite long.

That way, we clearly see that we deal with a special case for priting (I added a comment explaining this if someone works on that line again).