UserCmdGetBoxReceipt: remove unused parameter
UserCmdGetBoxReceipt: remove commented out condition
Merge pull request #397 from toxeus/develop

UserCmdGetBoxReceipt: remove confusions
Use SendMessage() call

OT still has some remnants of a SSL connection system that was removed in favor
of ZeroMQ. The OTServerConnection class does a lot of stuff that may have made
sense earlier but can be removed and simplified now.

This is the first step of a larger cleanup.

The old code had this repeated pattern

m_pClient->SetFocusToServerAndNym(*pServer, *pNym, m_pTransportCallback);
m_pClient->ProcessMessageOut(theMessage);
return m_pClient->CalcReturnVal(lRequestNumber);

and a variant that does not have last call to CalcReturnVal()

I replaced these two variants two atomic methods

* SendMessage(OTServerContract*, OTPseudonym*, OTMessage&, lRequestNumber)
* SendMessage(OTServerContract*, OTPseudonym*, OTmessage&)

The other place where SetFocusToServerAndNym is called now is in opentxs-script
which I will do separately.
Merge pull request #395 from OttoAllmendinger/pr-otapi-sendmessage

Use SendMessage() call
Fixed mixup between empty() and clear() so memory is realesed correctly
OTMessage: rename class to Message
OTMessage: rename files to Message.{h,c}pp
Merge pull request #398 from BugFreeSoftware/fix_cmd

Fixed mixup between empty() and clear() so memory is realesed correctly
Merge pull request #399 from toxeus/renameOTMessage

OTMessage: rename to Message
Some small comment cleanups
Remove -2 as an error code

The error code is never checked.

The methods for handling the error are not implemented.

The error is unlikely to occur.

The error can be prevented by using the correct data types.
Remove references to unused error code

Was always just demoted to -1 anyway

If we really care about request numbers > int32 we'll just use int64
Merge pull request #402 from OttoAllmendinger/pr-remove-minus2

Remove -2 error code
Use OTTransactionType::SetLoadInsecure() instead of manually setting m_bLoadSecurely.

SetLoadInsecure() exists for this purpose.
Cleanup in OTLedger::UpdateContents().

* Variable bSavingAbbreviated set redudantly.
* Removed nPartialACTUALCount that is only used for an assert that
  cannot fail.
* Make nPartialRecordCount depend on bSavingAbbreviated, instead of
  replicating the logic for both.
Merge pull request #404 from benma/develop

Cleanup in OTLedger::UpdateContents()
Merge OTMeCpp back into OT_ME
Merge pull request #401 from BugFreeSoftware/remove_otmecpp

Merge OTMeCpp back into OT_ME
Removed redundant m_strVersion = "2.0" statements.

It is already set in the base class OTContract.