From 74c70b74c2f42fe98d68d0fe9ea71db79baa7663 Mon Sep 17 00:00:00 2001 From: "M. J. Fromberger" Date: Tue, 5 Jul 2022 19:12:06 -0700 Subject: [PATCH] Remove check requiring Go 1.18. --- internal/mempool/v1/mempool.go | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/internal/mempool/v1/mempool.go b/internal/mempool/v1/mempool.go index d77cc23cb..4afd0509e 100644 --- a/internal/mempool/v1/mempool.go +++ b/internal/mempool/v1/mempool.go @@ -393,11 +393,12 @@ func (txmp *TxMempool) Update( newPreFn mempool.PreCheckFunc, newPostFn mempool.PostCheckFunc, ) error { - // Safety check: The caller is required to hold the lock. - if txmp.mtx.TryLock() { - txmp.mtx.Unlock() - panic("mempool: Update caller does not hold the lock") - } + // TODO(creachadair): This would be a nice safety check but requires Go 1.18. + // // Safety check: The caller is required to hold the lock. + // if txmp.mtx.TryLock() { + // txmp.mtx.Unlock() + // panic("mempool: Update caller does not hold the lock") + // } // Safety check: Transactions and responses must match in number. if len(blockTxs) != len(deliverTxResponses) { panic(fmt.Sprintf("mempool: got %d transactions but %d DeliverTx responses",