Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Superfluous rewriting of vector with beneficiaries #872

Open
afalaleev opened this issue Aug 12, 2019 · 0 comments
Open

Superfluous rewriting of vector with beneficiaries #872

afalaleev opened this issue Aug 12, 2019 · 0 comments
Labels
optional

Comments

@afalaleev
Copy link
Contributor

Description

golos.contracts/golos.publication/golos.publication.cpp

Line 207 in 22523f1

beneficiaries.reserve(benefic_map.size());

List of beneficiaries is already checked on duplicates:

golos.contracts/golos.publication/golos.publication.cpp

Line 188 in 22523f1

eosio::check(benefic_map.count(ben.account) == 0, "beneficiaries contain several entries for one account");

Solution
Remove creation of additional vector
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
optional
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@afalaleev