From a87ddeabdc17ea783171041fb20f8beec5bf221b Mon Sep 17 00:00:00 2001
From: Olaoluwa Osuntokun <laolu32@gmail.com>
Date: Fri, 12 Aug 2016 15:06:43 -0700
Subject: [PATCH] lnwallet: remove interaction with the btcutil.coinset package

This commit removes the wrapper functions used to rely on the coinset
package for coin selection. In a future commit the prior behavior will
be replaced by a custom coin selection implementation which estimates
the size of the funding transaction.
---
 lnwallet/coin_select.go | 73 -----------------------------------------
 1 file changed, 73 deletions(-)
 delete mode 100644 lnwallet/coin_select.go

diff --git a/lnwallet/coin_select.go b/lnwallet/coin_select.go
deleted file mode 100644
index e1d95491..00000000
--- a/lnwallet/coin_select.go
+++ /dev/null
@@ -1,73 +0,0 @@
-package lnwallet
-
-import (
-	"encoding/hex"
-
-	"github.com/roasbeef/btcd/btcjson"
-	"github.com/roasbeef/btcd/wire"
-	"github.com/roasbeef/btcutil"
-	"github.com/roasbeef/btcutil/coinset"
-)
-
-// lnCoin represents a single unspet output. Its purpose is to convert a regular
-// output to a struct adhering to the coinset.Coin interface
-type lnCoin struct {
-	hash     *wire.ShaHash
-	index    uint32
-	value    btcutil.Amount
-	pkScript []byte
-	numConfs int64
-	valueAge int64
-}
-
-func (l *lnCoin) Hash() *wire.ShaHash   { return l.hash }
-func (l *lnCoin) Index() uint32         { return l.index }
-func (l *lnCoin) Value() btcutil.Amount { return l.value }
-func (l *lnCoin) PkScript() []byte      { return l.pkScript }
-func (l *lnCoin) NumConfs() int64       { return l.numConfs }
-func (l *lnCoin) ValueAge() int64       { return l.valueAge }
-
-// Ensure lnCoin adheres to the coinset.Coin interface.
-var _ coinset.Coin = (*lnCoin)(nil)
-
-// newLnCoin creates a new "coin" from the passed output. Coins are required
-// in order to perform coin selection upon.
-func newLnCoin(output *btcjson.ListUnspentResult) (coinset.Coin, error) {
-	txid, err := wire.NewShaHashFromStr(output.TxID)
-	if err != nil {
-		return nil, err
-	}
-
-	pkScript, err := hex.DecodeString(output.ScriptPubKey)
-	if err != nil {
-		return nil, err
-	}
-
-	return &lnCoin{
-		hash: txid,
-		// btcjson.ListUnspentResult shows the amount in BTC,
-		// translate into Satoshi so coin selection can work properly.
-		value:    btcutil.Amount(output.Amount * 1e8),
-		index:    output.Vout,
-		pkScript: pkScript,
-		numConfs: output.Confirmations,
-		// TODO(roasbeef): output.Amount should be a int64, damn json-RPC :/
-		valueAge: output.Confirmations * int64(output.Amount),
-	}, nil
-}
-
-// outputsToCoins converts a slice of transaction outputs to a coin-selectable
-// slice of "Coins"s.
-func outputsToCoins(outputs []*btcjson.ListUnspentResult) ([]coinset.Coin, error) {
-	coins := make([]coinset.Coin, len(outputs))
-	for i, output := range outputs {
-		coin, err := newLnCoin(output)
-		if err != nil {
-			return nil, err
-		}
-
-		coins[i] = coin
-	}
-
-	return coins, nil
-}