BC break for v1.1.0

[afk11]

BC break for v1.1.0

[rubensayshi]

actual PR for using ints instead of floats; #58

[rubensayshi]

also I'd like to go and replace most places where we return false and throw exceptions.

and maybe add a custom exception class instead of \Exception?

[rubensayshi]

maybe you can make a v1.1 branch and already merge the stuff that's gonna go in there, then we can do PRs onto that instead of onto master and have less conflicts in the end?

[btcdrak]

well the right thing to do is create a 1.0 branch (dont use v prefix for branches) so we can maintain 1.0.x and the new stuff goes into master.

[rubensayshi]

okay, so @btcdrak @afk11 you guys agree to add a BitcoinLibException and start throwing it everywhere?

[btcdrak]

Generally a custom exception makes no sense. There are plenty semantic ones to choose in PHP.

[rubensayshi]

Okay, but replacing all the silent return false stuff with exceptions you agree with right?

[btcdrak]

@rubensayshi Yes: false should only be returned when a function processes something and returns false on valid input. If invalid inputs were received or something unexpected happened, it should always be an exception. This allows errors to bubble up the API. Returning false when something fails is definitely semantically wrong.

[afk11]

PR's #59 and #60 bring the library up to date with the latest phpecc commit and should be producing minimal pushdata ops.

I've added the throw exceptions issue to the list of tasks

@btcdrak we have a few releases along v1.0.x already. I'll create a branch called 1.0, tag it as v1.0.6, and when we merge these commits we'll start following 1.1.0

[rubensayshi]

Another PR to throw exceptions as much as possible; #61

[afk11]

Nice one! Looks like we have everything we wanted now. I'll merge these now and create one last branch of 1.0.

[afk11]

This has been merged and tagged as 1.1.0 https://github.com/Bit-Wasp/bitcoin-lib-php/releases/tag/1.1.0