coin-grinder missing test for TOTAL_TRIES #33419

issue yancyribbens openend this issue on September 17, 2025
  1. yancyribbens commented at 4:24 pm on September 17, 2025: contributor

    The following (mutation) code can be remove from coin-grinder here without encountering a test failure

    0if (curr_try >= TOTAL_TRIES) {
    1    // Solution is not guaranteed to be optimal if `curr_try` hit TOTAL_TRIES
    2    result.SetAlgoCompleted(false);
    3    break;
    4}
    

    This ought to be a similar test to Test BnB attempt limit (TOTAL_TRIES)

  2. yancyribbens commented at 4:26 pm on September 17, 2025: contributor
  3. fanquake added the label Tests on Sep 18, 2025
  4. brunoerg commented at 12:42 pm on September 18, 2025: contributor

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-09-26 15:13 UTC

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