Concept ACK

The new tests look good and logically suffices with the code of both the functions.

I would like to propose some additions to the test. I am not yet proficient in writing tests for functions, so I might be wrong. I was understanding the ParseScript function’s code, and I think it would be good to do some additions to the testing of this function

• BOOST_CHECK_EQUAL(HexStr(ParseScript("-9")), "59");
  To test the line:
  (w->front() == '-' && w->size() > 1 && std::all_of(w->begin()+1, w->end(), ::IsDigit)))

And,

• BOOST_CHECK_EXCEPTION(ParseScript("11111111111111111111111"), std::runtime_error, HasReason("script parse error: decimal numeric value only allowed in the range -0xFFFFFFFF...0xFFFFFFFF"));
  To test the line:
  throw std::runtime_error("script parse error: decimal numeric value only allowed in the " "range -0xFFFFFFFF...0xFFFFFFFF");

I hope my suggestion might help you improve upon the PR.

Updates since my last review:

1. New test lines have been added to the test of the ParseScript and ParseMoney functions.

The new checks work correctly and compile successfully. I would like to suggest one more addition, which I somehow missed when I reviewed this PR earlier.

In the ParseScript function:

boost::algorithm::split(words, s, boost::algorithm::is_any_of(" \t\n"), boost::algorithm::token_compress_on);

    for (std::vector<std::string>::const_iterator w = words.begin(); w != words.end(); ++w)
    {
        …
    }

That means it can work for multiple integer values given to it. But there is no explicit check for it. Some further checks like:
BOOST_CHECK_EQUAL(HexStr(ParseScript("1 2")), "5152");
&
BOOST_CHECK_EQUAL(HexStr(ParseScript("1 -9")), "510189");

Should be added.

tACK fa1477e

Updates since my last review:

1. Test for multiple integer input in one string is added to ParseScript tests.