[bug?] New wallet version

[pbies]

Reproduce:

1. create wallet (not descriptor wallet, empty), version created is noted as 10500 in debug.log
2. close the wallet
3. open it once again
4. debug.log says version 169900 for this wallet

[achow101]

That is expected behavior.

A newly created wallet will initially have the version of 10500 before it is subsequently set to the current latest of 169900. The logging happens before that update.

[NotATether]

A newly created wallet will initially have the version of 10500 before it is subsequently set to the current latest of 169900. The logging happens before that update.

Are there plans to introduce a second debug message after the update that says something like "Wallet <xyz> has been changed from version 123 to 456"?

[achow101]

Are there plans to introduce a second debug message after the update that says something like "Wallet <xyz> has been changed from version 123 to 456"?

PRs welcome.

[NotATether]

Are there plans to introduce a second debug message after the update that says something like "Wallet <xyz> has been changed from version 123 to 456"?

PRs welcome.

Pull request submitted.

[NotATether]

@achow101:

I'm not familiar with Bitcoin Core's PR merging process. How long does it take on average for somebody to review a commit and [N]ACK it?

[yes, I double posted 20 hours later, but I didn't want to leave this until 23:30 UTC+3 because I'll probably be asleep by then]

[achow101]

I'm not familiar with Bitcoin Core's PR merging process. How long does it take on average for somebody to review a commit and [N]ACK it?

Depends on how complicated and how many people care. It can be anywhere from a few hours to a few years.

[NotATether]

Are there plans to introduce a second debug message after the update that says something like "Wallet <xyz> has been changed from version 123 to 456"?

PRs welcome.

Pull request submitted.

This PR has been merged so this thread can be locked now.