fix non-standard reason string in main.cpp

[Diapolo]

No description provided.

[BitcoinPullTester]

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/bbfce8a4c9e7541d06dcc857dd0a82ec14d92180 for binaries and test log.
This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/
Contact BlueMatt on freenode if something looks broken.

[jgarzik]

I cannot discern from your commit message, or the patch, why this is a "fix", or is needed at all.

[Diapolo]

https://github.com/bitcoin/bitcoin/blob/master/src/main.cpp#L438
https://github.com/bitcoin/bitcoin/blob/master/src/main.cpp#L442

[laanwj]

@Diaplo Can you link to the standard that would make this nonstandard? I cannot find any of these reasons in BIP0022 in a quick glance.

[petertodd]

ACK

Good to be clear as to where the non-standardness is given scriptPubKeys are allowed to have non-standard pushdata's. If that changes in the future we can create an additional scriptPubKey-non-canonical-push message.

[Diapolo]

See above links, I just used the wording that WAS already used for the other reason strings... I don't claim there is a BIP for this :-P.

[sipa]

ACK