Peerplays_Blockchain/peerplays_migrated
1
0
Fork 0
Code Issues 35 Pull requests 4 Projects Releases 20 Packages Wiki Activity Actions

remove htonl from memo checksum when it appears to provide no obvious benefit

Browse source
This commit is contained in:
Daniel Larimer 2015-07-02 10:04:07 -04:00
parent fefa0f65f8
commit d66ee4c8cc
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
libraries/chain/operations.cpp
View file

@ -876,7 +876,7 @@ share_type worker_create_operation::calculate_fee(const fee_schedule_type& k) co
string memo_message::serialize() const
{
auto serial_checksum = string(sizeof(checksum), ' ');
(uint32_t&)(*serial_checksum.data()) = htonl(checksum);
(uint32_t&)(*serial_checksum.data()) = checksum;
return serial_checksum + text;
}
@ -884,7 +884,7 @@ memo_message memo_message::deserialize(const string& serial)
{
memo_message result;
FC_ASSERT( serial.size() >= sizeof(result.checksum) );
result.checksum = ntohl((uint32_t&)(*serial.data()));
result.checksum = ((uint32_t&)(*serial.data()));
result.text = serial.substr(sizeof(result.checksum));
return result;
}