Skip to content

Commit

Permalink
client/asset/eth: add (*assetWallet).amtString method
Browse files Browse the repository at this point in the history
  • Loading branch information
chappjc committed Feb 16, 2023
1 parent a9fdf04 commit bccabb8
Showing 1 changed file with 16 additions and 8 deletions.
24 changes: 16 additions & 8 deletions client/asset/eth/eth.go
Original file line number Diff line number Diff line change
Expand Up @@ -431,10 +431,10 @@ type baseWallet struct {
// assetWallet satisfies the dex.Wallet interface.
type assetWallet struct {
*baseWallet
assetID uint32
tipChange func(error)
log dex.Logger
atomicUnit string
assetID uint32
tipChange func(error)
log dex.Logger
ui dex.UnitInfo

lockedFunds struct {
mtx sync.RWMutex
Expand Down Expand Up @@ -681,7 +681,7 @@ func NewWallet(assetCFG *asset.WalletConfig, logger dex.Logger, net dex.Network)
contractors: make(map[uint32]contractor),
evmify: dexeth.GweiToWei,
atomize: dexeth.WeiToGwei,
atomicUnit: dexeth.UnitInfo.AtomicUnit,
ui: dexeth.UnitInfo,
maxSwapsInTx: gasCeil / maxProportionOfBlockGasLimitToUse / maxSwapGas,
maxRedeemsInTx: gasCeil / maxProportionOfBlockGasLimitToUse / maxRedeemGas,
}
Expand Down Expand Up @@ -963,7 +963,7 @@ func (w *ETHWallet) OpenTokenWallet(tokenCfg *asset.TokenConfig) (asset.Wallet,
contractors: make(map[uint32]contractor),
evmify: token.AtomicToEVM,
atomize: token.EVMToAtomic,
atomicUnit: token.UnitInfo.AtomicUnit,
ui: token.UnitInfo,
maxSwapsInTx: gasCeil / maxProportionOfBlockGasLimitToUse / maxSwapGas,
maxRedeemsInTx: gasCeil / maxProportionOfBlockGasLimitToUse / maxRedeemGas,
}
Expand Down Expand Up @@ -994,6 +994,10 @@ func (eth *baseWallet) OwnsDepositAddress(address string) (bool, error) {
return addr == eth.addr, nil
}

func (w *assetWallet) amtString(amt uint64) string {
return fmt.Sprintf("%s %s", w.ui.ConventionalString(amt), w.ui.Conventional.Unit)
}

// fundReserveType represents the various uses for which funds need to be locked:
// initiations, redemptions, and refunds.
type fundReserveType uint32
Expand Down Expand Up @@ -1042,7 +1046,7 @@ func (w *assetWallet) lockFunds(amt uint64, t fundReserveType) error {
if balance.Available < amt {

return fmt.Errorf("attempting to lock more %s for %s than is currently available. %d > %d %s",
dex.BipIDSymbol(w.assetID), t, amt, balance.Available, w.atomicUnit)
dex.BipIDSymbol(w.assetID), t, amt, balance.Available, w.ui.AtomicUnit)
}

w.lockedFunds.mtx.Lock()
Expand Down Expand Up @@ -1340,6 +1344,8 @@ func (w *ETHWallet) FundOrder(ord *asset.Order) (asset.Coins, []dex.Bytes, error
// some work for the caller as well. We can't just always do it that way and
// remove RedeemN, since we can't guarantee that the redemption asset is in
// our fee-family. though it could still be an AccountRedeemer.
w.log.Debugf("Locking %s to swap %s in up to %d swaps at a fee rate of %d gwei/gas using up to %d gas per swap",
w.amtString(ethToLock), w.amtString(ord.Value), ord.MaxSwapCount, ord.MaxFeeRate, g.Swap)

coin := w.createFundingCoin(ethToLock)

Expand Down Expand Up @@ -1381,6 +1387,8 @@ func (w *TokenWallet) FundOrder(ord *asset.Order) (asset.Coins, []dex.Bytes, err
}
}()

w.log.Debugf("Locking %s to swap %s in up to %d swaps at a fee rate of %d gwei/gas using up to %d gas per swap",
w.parent.amtString(ethToLock), w.amtString(ord.Value), ord.MaxSwapCount, ord.MaxFeeRate, g.Swap)
if err := w.parent.lockFunds(ethToLock, initiationReserve); err != nil {
return nil, nil, err
}
Expand Down Expand Up @@ -2635,7 +2643,7 @@ func (w *TokenWallet) canSend(value uint64, isPreEstimate bool) (uint64, *big.In
}
avail := bal.Available
if avail < value {
return 0, nil, fmt.Errorf("not enough tokens: have %[1]d %[3]s need %[2]d %[3]s", avail, value, w.atomicUnit)
return 0, nil, fmt.Errorf("not enough tokens: have %[1]d %[3]s need %[2]d %[3]s", avail, value, w.ui.AtomicUnit)
}

ethBal, err := w.parent.Balance()
Expand Down

0 comments on commit bccabb8

Please sign in to comment.