feature:automatically include the elapsed time in logging #32925

pull marbcGH wants to merge 1 commits into bitcoin:master from marbcGH:master changing 1 files +1 −5
  1. marbcGH commented at 0:43 am on July 9, 2025: none

    Simple, minimal change to improve logging by including the elapsed time in the “completed” log message even when msg_on_completion is set to true. Motivation is to contribute, and improve Bitcoin Core user experience & Bitcoin Core developer experience with a simple log modification.

    Current log outputs:

    0LoadBlockIndex completed
    

    After modification log output:

    0LoadBlockIndex: LoadBlockIndex completed (452.87ms)
    

    Current state doesn’t include how long the operation took. Which can force developers to either Parse “started” and “completed” lines separately Or add an intermediate log themselves to capture the duration.

  2. feature:automatically include the elapsed time 1fb9fca3e4
  3. DrahtBot commented at 0:43 am on July 9, 2025: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32925.

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

  4. maflcko closed this on Jul 9, 2025


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2025-07-11 09:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me