Bug in parseBase for numbers containing a dot ('0.0079')

[axic]

Bug in parseBase for numbers containing a dot ('0.0079')

[indutny]

It throws now.

[axic]

The same applies to parseHex (as basically it is the same code as parseBase, with one exception: range is limited to f/F).

new BN('1234', 16) // <BN: 1234>
new BN('12.34', 16) // <BN: 12e34>

[indutny]

I know that it should throw, but it doesn't right now. The thing is that this is an big integer library, so you should not really pass data like this to it.

[axic]

I do know that and I don't pass those kind of numbers. However, users of my library are. I can either do pre-processing there which is quite big of an overhead. I would prefer bn.js to throw, I have the PR ready, will submit it soon - it is quite small.

[axic]

A note on the integer side, the library's name is short for big number and doesn't specifically mentions anywhere it is integers and not decimals. I know that, but it is not clear from the first sight at the README, maybe it should be updated.

[indutny]

Yeah, I totally agree. It should be updated. Thanks!

[fanatid]

new BN(4.2) return <BN: 4>
also should be assert here

[dcousens]

A note was added to the README at some point

Note: decimals are not supported in this library.