Skip to content

Conversation

@Diapolo
Copy link

@Diapolo Diapolo commented Jan 8, 2015

No description provided.

@laanwj
Copy link
Member

laanwj commented Jan 8, 2015

ACK on d9727fb 4000726 4fbfbf9
NACK on 0134b6b, the change in CDBEnv::CloseDb is unnecessary, and the rest of the changes are just some idle whitespace changes and move-declarations-around.

@Diapolo
Copy link
Author

Diapolo commented Jan 8, 2015

@laanwj Removed number 4.

@jonasschnelli jonasschnelli mentioned this pull request Jan 8, 2015
@laanwj
Copy link
Member

laanwj commented Jan 20, 2015

Now that the newwallet work is starting (#5686). I want to discourage any changes to the old wallet code not strictly necessary. So I'm closing this, sorry.

@laanwj laanwj closed this Jan 20, 2015
@Diapolo
Copy link
Author

Diapolo commented Jan 20, 2015

Yeah nothing new...

@Diapolo Diapolo deleted the db branch January 20, 2015 19:40
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants