Sema: check that nr is integer

[wmertens]

I mistakenly assumed that 0.5 rps would work…

[OlliV]

If nr would be set to null this would happen: null | 0 === 0

[OlliV]

Suggested change

	if ((nr | 0) !== nr) {
	if (nr && !Number.isInteger(nr)) {

[wmertens]

actually, nr should be a positive non-zero integer, so how about if (!nr || nr < 1 || (nr | 0) !== nr)?

Number.isInteger doesn't exist in IE, but maybe that's not an issue?

[OlliV]

Good point, that seems good to me!

Oh no, I didn't thought about that. The polyfill is actually quite simple:

Number.isInteger = Number.isInteger || function(value) {
  return typeof value === 'number' && 
    isFinite(value) && 
    Math.floor(value) === value;
};

Of course we shouldn't actually polyfill it but maybe just do these checks in the if clause.

Number.isFinite() seems to have an implemented in all browsers.