Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Remove dead code from fork_database #7101

Conversation

hqueue
Copy link

@hqueue hqueue commented Apr 10, 2019

Change Description

Remove following dead code from fork_database

  • fork_database::add( const header_confirmation& c )
  • fork_database::set_bft_irreversible( block_id_type id )

Above two member functions of fork_datatbase are not used anymore since PR #6149 which removed controller::push_confirmation.

Closes #7100

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

Remove following dead code from fork_database

- fork_database::add( const header_confirmation& c )
- fork_database::set_bft_irreversible( block_id_type id )

Closes EOSIO#7100

Signed-off-by: Hyung-Kyu Choi <[email protected]>
@heifner
Copy link
Contributor

heifner commented Apr 10, 2019

@arhag I assume you have already made these changes on your fork db refactor branch.

@arhag
Copy link
Contributor

arhag commented Apr 10, 2019

@heifner: Yes, this dead code and more was removed in PR #6624.

I'm closing this PR since these changes will be incorporated into develop when the forced-replay branch is merged into develop.

@arhag arhag closed this Apr 10, 2019
@heifner
Copy link
Contributor

heifner commented Apr 10, 2019

Thanks @hqueue for your contribution.

@hqueue
Copy link
Author

hqueue commented Apr 10, 2019

@heifner Got it. Thanks for the information and please also close issue #7100 if it is duped too :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants