Skip to content

init: clean up load chainstate function#240

Open
Ataraxia009 wants to merge 1 commit into
bitcoinknots:29.x-knotsfrom
Ataraxia009:clean-up-load-chainstate
Open

init: clean up load chainstate function#240
Ataraxia009 wants to merge 1 commit into
bitcoinknots:29.x-knotsfrom
Ataraxia009:clean-up-load-chainstate

Conversation

@Ataraxia009

@Ataraxia009 Ataraxia009 commented Nov 26, 2025

Copy link
Copy Markdown

Cleaning up LoadChainTip function, had some unnecessary/redundant lines of code, more specifically:

  • removing repetitive call to GetBestBlock()
  • removing extra call to tip = m_chain.Tip();
  • Changing call from LogPrintf to LogInfo

@Ataraxia009 Ataraxia009 marked this pull request as ready for review November 26, 2025 18:02

@bigshiny90 bigshiny90 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Very straight-forward and logical.

ACK 923c3ab

@mhrussell1 mhrussell1 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good. Sorry don't know how to ACK,

Retropex pushed a commit to Retropex/bitcoin that referenced this pull request Mar 23, 2026
…451f

1868a84451f Merge bitcoin-core/libmultiprocess#245: type-context.h: Extent cancel_mutex lock to prevent theoretical race
fd4a90d3103 Merge bitcoin-core/libmultiprocess#244: ci: suppress two tidy lint issues
16dfc368640 ci: avoid bugprone-unused-return-value lint in test
dacd5eda464 ci: suppress nontrivial-threadlocal lint in proxy.cpp
ef96a5b2be2 doc: Comment cleanups after bitcoinknots#240
e0f1cd76219 type-context.h: Extent cancel_mutex lock to prevent theoretical race
290702c74ce Merge bitcoin-core/libmultiprocess#240: Avoid errors from asynchronous (non-c++) clients
3a69d4755af Merge bitcoin-core/libmultiprocess#241: doc: Bump version number v7 -> v8
0174450ca2e Prevent crash on unclean disconnect if abandoned IPC call returns interface pointer
ddb5f74196f Allow simultaneous calls on same Context.thread
c4762c7b513 refactor: Add ProxyServer<Thread>::post() method
0ade1b40ac5 doc: Bump version number

git-subtree-dir: src/ipc/libmultiprocess
git-subtree-split: 1868a84451fe1b6a00116375a5f717230bb2533e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants