I have made some changes to base_convert which I feel would be more
consistent with the current PHP (warning when there are errors, and not
just returning the best value it can)
The RFC has some examples of what will change.
Currently the code works but a lot of tests fail due to extreme range's
(also mentioned in the RFC)
If this passes I will fix the effected tests and add some more covering
the new behavior.
Let me know your thoughts
I definitely agree with the part of the RFC that warns on garbage
characters in the string. I'm not so sure about the changes in sign
handling. The problem I see is that certain bases (in particular hex) are
pretty much never used with an explicit sign, instead they are understood
to be in two's complement representation.
For example, code like this will currently work:
// string(16) "ffffffff00000000"
While after your change it will result in "-100000000", which for hex
numbers is rather unexpected. (This example is somewhat non-great because
there is a loss of accuracy for 64bit numbers represented as doubles -- the
same examples with 32-bit integers on 32-bit systems would be fully