cppcheck: fix redundantAssignment
Merge pull request #100 from toxeus/fixRedundantAssignment

cppcheck: fix redundantAssignment
Removed useless copy/paste comments before every OT_CLI_GetValueByKey()
Removed useless windows batch file
Updates scripts readme with new opentxs CLI
Removed setting of default mypurse when myacct is specified
Removed setting of default hispurse when hisacct is specified
Added lots of negative tests to ot_test.opentxs
Changed return type of show_mail_message to bool instead of int32_t
Changed return type of show_outmail_message to bool instead of int32_t
OTAPI_Exec::GetNym_MailCount returns error value for unknown nym
OTAPI_Exec::GetNym_OutmailCount returns error value for unknown nym
OTAPI_Exec::GGetNym_OutpaymentsCount returns error value for unknown nym
Change error message for VerifyExists
Give unified message when box is empty for mail, outmail, and outpayment
Fixed check for message count in box for mail, outmail, and outpayment
Also unified the accompanying error message
Added check for missing myacct to main_transfer
Added checks for missing server, mynym, and hisnym parameters to main_sendmsg
Also removed useless FYI from usage
Fixed main_sendcash to check for valid recipient
Moved this check so it will be done before trying to do cash withdrawal
Added check on message index for mail, outmail and deloutmail
Also reformatted mail and outmail to become clearer
Some reformatting and removing useless comments
load_or_retrieve_mint did not check the nym id.