Kaspars

Results 23 comments of Kaspars

[Runnable example](https://ideone.com/D5RwNF) `"m"` isnt a valid argument. What output do you expect? Can either: 1. Throw 2. Cast to 0 I'm leaning mostly towards (1)

Good idea testing against python. But I dont want to add another language in the build script.

Should be safe to change to `long long`.

I dont like the current solution to dividing by zero. Throwing an exception might be acceptable, so the behaviour is the same as with int type.

@karlphillip Before going over any PR's I want to add an automated test suite to reduce regressions. Havent had any time lately; not sure if this is used in production...

If you feel like it then sure, go ahead. I'd appreciate it.

Sounds good. But make it a private method. I want to keep the public api simple.

Multiplication seems produce incorrect results in some cases, I'll go over it later today (or tomorrow, depending on the timezone)

Can you give an example of when it produces an incorrect result?