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

use assert instead of checking condition with low possibility

Browse source
This commit is contained in:
gladcow 2019-11-11 13:13:57 +03:00
parent 62f973ca80
commit dbf6be02c5
1 changed files with 2 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
libraries/wallet/wallet.cpp
View file

@ -1940,8 +1940,7 @@ public:
owners.resize(son_objects.size()); owners.resize(son_objects.size());
std::transform(son_objects.begin(), son_objects.end(), owners.begin(), std::transform(son_objects.begin(), son_objects.end(), owners.begin(),
[](const fc::optional<son_object>& obj) { [](const fc::optional<son_object>& obj) {
if(!obj) FC_ASSERT(obj, "Invalid active SONs list in global properties.");
FC_THROW("Invalid active SONs list in global properties.");
return obj->son_account; return obj->son_account;
}); });
vector<fc::optional<account_object>> accs = _remote_db->get_accounts(owners); vector<fc::optional<account_object>> accs = _remote_db->get_accounts(owners);
@ -1949,8 +1948,7 @@ public:
std::transform(accs.begin(), accs.end(), gpo.active_sons.begin(), std::transform(accs.begin(), accs.end(), gpo.active_sons.begin(),
std::inserter(result, result.end()), std::inserter(result, result.end()),
[](fc::optional<account_object>& acct, son_id_type& sid) { [](fc::optional<account_object>& acct, son_id_type& sid) {
if(!acct) FC_ASSERT(acct, "Invalid active SONs list in global properties.");
FC_THROW("Invalid active SONs list in global properties.");
return std::make_pair<string, son_id_type>(string(acct->name), std::move(sid)); return std::make_pair<string, son_id_type>(string(acct->name), std::move(sid));
}); });
return result; return result;