refactor: Remove SignetTxs::m_valid and use optional instead

m_valid implies the block solution has been checked, which is not the
case. It only means the txs could be parsed. C++17 comes with
std::optional, so just use that instead.

1 files changed, 1 insertions, 1 deletions

diff --git a/src/test/fuzz/signet.cpp b/src/test/fuzz/signet.cpp
index 4736ae27f5..135bdbd1a0 100644
--- a/src/test/fuzz/signet.cpp
+++ b/src/test/fuzz/signet.cpp
@@ -29,6 +29,6 @@ void test_one_input(const std::vector<uint8_t>& buffer)
     }
     (void)CheckSignetBlockSolution(*block, Params().GetConsensus());
     if (GetWitnessCommitmentIndex(*block) != NO_WITNESS_COMMITMENT) {
-        (void)SignetTxs(*block, ConsumeScript(fuzzed_data_provider));
+        (void)SignetTxs::Create(*block, ConsumeScript(fuzzed_data_provider));
     }
 }