refactor: context first in argument list
(merely by convention) License: MIT Signed-off-by: Brian Tiger Chow <brian@perfmode.com>
Brian Tiger Chow committed
Dec 16, 2014 at 20:52 UTC
10e970c0cf0a4d90d9164456d9d8dc5da115a193
3 files changed
+4
-3
exchange/bitswap/bitswap.go
+1
-1
@@ -56,7 +56,7 @@ func New(parent context.Context, p peer.Peer, network bsnet.BitSwapNetwork, rout
56
blockstore: bstore,
57
cancelFunc: cancelFunc,
58
notifications: notif,
59
- ledgermanager: strategy.NewLedgerManager(bstore, ctx),
59
+ ledgermanager: strategy.NewLedgerManager(ctx, bstore),
60
routing: routing,
61
sender: network,
62
wantlist: wl.New(),
exchange/bitswap/strategy/ledgermanager.go
+1
-1
@@ -34,7 +34,7 @@ type LedgerManager struct {
34
workSignal chan struct{}
35
}
36
37
-func NewLedgerManager(bs bstore.Blockstore, ctx context.Context) *LedgerManager {
37
+func NewLedgerManager(ctx context.Context, bs bstore.Blockstore) *LedgerManager {
38
lm := &LedgerManager{
39
ledgerMap: make(ledgerMap),
40
bs: bs,
exchange/bitswap/strategy/ledgermanager_test.go
+2
-1
@@ -23,7 +23,8 @@ func newPeerAndLedgermanager(idStr string) peerAndLedgermanager {
23
return peerAndLedgermanager{
24
Peer: testutil.NewPeerWithIDString(idStr),
25
//Strategy: New(true),
26
- ls: NewLedgerManager(blockstore.NewBlockstore(sync.MutexWrap(ds.NewMapDatastore())), context.TODO()),
26
+ ls: NewLedgerManager(context.TODO(),
27
+ blockstore.NewBlockstore(sync.MutexWrap(ds.NewMapDatastore()))),
28
}
29
}
30