Skip to content

dgb: correct won_share_inputs header drift -- m_on_block_found bound, not a nullopt stub - #437

Merged
frstrtr merged 1 commit into
masterfrom
dgb/won-share-inputs-doc-drift
Jun 24, 2026
Merged

dgb: correct won_share_inputs header drift -- m_on_block_found bound, not a nullopt stub#437
frstrtr merged 1 commit into
masterfrom
dgb/won-share-inputs-doc-drift

Conversation

@frstrtr

@frstrtr frstrtr commented Jun 24, 2026

Copy link
Copy Markdown
Owner

What

Comment-only correction to src/impl/dgb/coin/won_share_inputs.hpp. The banner described ShareTracker::m_on_block_found as "today an interim nullopt stub in main_dgb.cpp". That is stale.

main_dgb.cpp now binds m_on_block_found = dgb::coin::make_on_block_found(...) -- the #82 dual-path broadcaster (P2P-relay arm + external-digibyted submitblock fallback), proven end-to-end on regtest (both arms accept a tx-bearing won block).

Scope

  • Comment-only. Zero code/behavior delta.
  • No consensus, share-format, PPLNS, or p2pool-merged-v36-surface change.
  • Per-coin isolation preserved: src/impl/dgb/ only.
  • The adjacent "remaining three inputs" paragraph is left intact -- it accurately describes this seam pulling only the two share-carried fields (small_header, merkle_link); the other three are supplied template-side in reconstruct_closure.hpp.

Fenced/additive doc fix. No self-merge.

…und, not a nullopt stub

The won_share_inputs.hpp banner described ShareTracker::m_on_block_found as
"today an interim nullopt stub in main_dgb.cpp". That is stale: main_dgb.cpp
now binds m_on_block_found = dgb::coin::make_on_block_found(...) -- the #82
dual-path broadcaster (P2P-relay arm + external-digibyted submitblock
fallback), proven end-to-end on regtest. Comment-only; no code, behavior,
consensus, share-format, or p2pool-merged-v36-surface change.
@frstrtr
frstrtr merged commit 359f071 into master Jun 24, 2026
32 checks passed
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.

1 participant