From 2ae1b7dbbe5750acc46ed43e354d96cbe7977ccb Mon Sep 17 00:00:00 2001 From: "Johan T. Halseth" Date: Mon, 5 Feb 2018 17:26:35 -0500 Subject: [PATCH] contractcourt: remove TODO for checking double spends from PublishTx --- contractcourt/channel_arbitrator.go | 1 - contractcourt/contract_resolvers.go | 1 - 2 files changed, 2 deletions(-) diff --git a/contractcourt/channel_arbitrator.go b/contractcourt/channel_arbitrator.go index eb786d49..668309c1 100644 --- a/contractcourt/channel_arbitrator.go +++ b/contractcourt/channel_arbitrator.go @@ -449,7 +449,6 @@ func (c *ChannelArbitrator) stateStep(bestHeight uint32, bestHash *chainhash.Has // At this point, we'll now broadcast the commitment // transaction itself. if err := c.cfg.PublishTx(closeTx); err != nil { - // TODO(roasbeef): need to check for errors (duplicate) log.Errorf("ChannelArbitrator(%v): unable to broadcast "+ "close tx: %v", c.cfg.ChanPoint, err) return StateError, closeTx, err diff --git a/contractcourt/contract_resolvers.go b/contractcourt/contract_resolvers.go index 2fa4e2c7..5cc9ffa6 100644 --- a/contractcourt/contract_resolvers.go +++ b/contractcourt/contract_resolvers.go @@ -582,7 +582,6 @@ func (h *htlcSuccessResolver) Resolve() (ContractResolver, error) { // // TODO(roasbeef): after changing sighashes send to tx bundler if err := h.PublishTx(h.htlcResolution.SignedSuccessTx); err != nil { - // TODO(roasbeef): detect double spends return nil, err }