Skip to content

Conversation

@furszy
Copy link

@furszy furszy commented Sep 30, 2019

Back port from bitcoin#10952.

This PR is on top of #1029 because of both PRs modifying init.cpp.
Check only the last commit.

* OS memory allocation fail handler.
* OS signal handler registration method created to remove code duplication.
* AppInitBasicSetup() method created, organizing better the setup step of the wallet initialization.
 * Stop loading block indexes on wallet startup if shutdown was requested.
 * Wallet loading, wallet rescan and block index load time logged in a more understandable way.
@random-zebra random-zebra added Backport Startup Wallet startup changes and/or improvements Wallet labels Sep 30, 2019
@random-zebra random-zebra added this to the 4.0.0 milestone Sep 30, 2019
…emove vchDefaultKey and have better first run detection".

[Wallet][Startup][DB][Backport] Don't create any default address
@furszy furszy force-pushed the 2019_remove_vchDefaultKey branch from 0ca58ef to 6637fd1 Compare September 30, 2019 00:53
Copy link

@random-zebra random-zebra left a comment

Choose a reason for hiding this comment

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

utACK

Copy link

@random-zebra random-zebra left a comment

Choose a reason for hiding this comment

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

ACK 83d8d00

Copy link

@Warrows Warrows left a comment

Choose a reason for hiding this comment

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

ACK 83d8d00

Copy link
Collaborator

@Fuzzbawls Fuzzbawls left a comment

Choose a reason for hiding this comment

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

ACK 83d8d00

random-zebra added a commit that referenced this pull request Oct 11, 2019
…m wallet.h

83d8d00 [Trivial] Init.cpp, instead of use LogPrint, use error. (furszy)
6637fd1 [Wallet][Startup][DB][Backport] bitcoin#10952 BTC back port. Named "Remove vchDefaultKey and have better first run detection". (furszy)

Pull request description:

  Back port from bitcoin#10952.

  This PR is on top of #1029 because of both PRs modifying `init.cpp`.
  Check only the last commit.

ACKs for top commit:
  random-zebra:
    ACK 83d8d00
  Warrows:
    ACK 83d8d00
  Fuzzbawls:
    ACK 83d8d00

Tree-SHA512: 0e05f7f80897a421c541580abbd10af37db083a43c6edf49f61e8d1977c58c6e486b1f17e0eac4fb4e8257200eea3e04b5510321cb4484e110f466f5a5132151
@random-zebra random-zebra merged commit 83d8d00 into PIVX-Project:master Oct 11, 2019
@furszy furszy deleted the 2019_remove_vchDefaultKey branch November 29, 2022 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backport Startup Wallet startup changes and/or improvements Upstream Wallet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants