Skip to content

Conversation

dhruv
Copy link
Contributor

@dhruv dhruv commented Apr 30, 2021

Copy link
Member

@maflcko maflcko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for not backlinking, but another pull has those already.

@@ -182,7 +182,7 @@ class CBlockIndex
//!
//! Note: this value is modified to show BLOCK_OPT_WITNESS during UTXO snapshot
//! load to avoid the block index being spuriously rewound.
//! @sa RewindBlockIndex
//! @sa NeedsRedownload
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How is this change different from fa49430 in #21789 ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not different at all :)

@dhruv
Copy link
Contributor Author

dhruv commented Apr 30, 2021

Changes addressed in #21789 as @MarcoFalke pointed out. Closing this PR.

@dhruv dhruv closed this Apr 30, 2021
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants