Platform: Code4rena
Start Date: 14/07/2022
Pot Size: $25,000 USDC
Total HM: 2
Participants: 63
Period: 3 days
Judge: PierrickGT
Total Solo HM: 1
Id: 147
League: ETH
Rank: 41/63
Findings: 1
Award: $39.09
🌟 Selected for report: 0
🚀 Solo Findings: 0
🌟 Selected for report: hickuphh3
Also found by: 0x29A, 0x52, 0xNazgul, Chom, Deivitto, ElKu, Funen, IllIllI, Meera, ReyAdmirado, SooYa, TomJ, Trumpero, Waze, __141345__, ak1, asutorufos, c3phas, cRat1st0s, csanuragjain, delfin454000, exd0tpy, fatherOfBlocks, hake, hansfriese, horsefacts, hyh, karanctf, kenzo, kyteg, ladboy233, pashov, peritoflores, rajatbeladiya, rbserver, reassor, rokinot, simon135, wastewa
39.0864 USDC - $39.09
The same typos (overriden
and specialised
) occur in all three lines referenced below:
/// Useful as a method so it can be overriden by specialised witches that may need to do extra accounting or notify 3rd parties
Change overriden by specialised
to overridden by specialized
in each case. This assumes that a decision has been made to use standard American English throughout Yield Witch v2.
/// @dev Calculates the auction initial values, the 2 non-trivial values are how much art must be repayed
Change repayed
to repaid
/// @dev transfers funds from the ilkJoin to the liquidator (and potentially the auctioneer if they're differente people)
Change differente
to different
/// @dev quoutes hoy much ink a liquidator is expected to get if it repays an `artIn` amount
Change quoutes hoy
to quotes how
For the sake of readability, long comments over 79 characters should wrap using multi-line comment syntax
Examples
Below are ten of the longest comments whose readability could be improved by converting them to multi-line comments:
/// Useful as a method so it can be overriden by specialised witches that may need to do extra accounting or notify 3rd parties
Recommendation:
/// Useful as a method so it can be overridden by specialized witches /// that may need to do extra accounting or notify 3rd parties.
// We store the proportion of the vault to auction, which is the whole vault if the debt would be below dust.
Recommendation:
// We store the proportion of the vault to auction, which is the whole vault // if the debt would be below dust.
/// @return auctioneerCut Amount paid to whomever started the auction. 0 if it's the same address that's calling this method
Recommendation:
/// @return auctioneerCut Amount paid to whomever started the auction: /// 0 if it's the same address that's calling this method.
/// @dev Pay up to `maxArtIn` debt from a vault in liquidation using fyToken, getting at least `minInkOut` collateral.
Recommendation:
/// @dev Pay up to `maxArtIn` debt from a vault in liquidation using fyToken, /// getting at least `minInkOut` collateral.
/// @dev transfers funds from the ilkJoin to the liquidator (and potentially the auctioneer if they're differente people)
Recommendation:
/// @dev transfers funds from the ilkJoin to the liquidator /// (and potentially the auctioneer if they're different people).
/// @notice Update accounting on the Witch and on the Cauldron. Delete the auction and give back the vault if finished.
Recommendation:
/// @notice Update accounting on the Witch and on the Cauldron. Delete the auction /// and give back the vault if finished.
/// This function doesn't verify the vaultId matches the vault and auction passed. Check before calling.
Recommendation:
/// This function doesn't verify that the vaultId matches the vault /// and the auction has passed. Check both before calling.
/// @return auctioneerCut How much collateral the auctioneer is expected to get. 0 if liquidator == auctioneer
Recommendation:
/// @return auctioneerCut How much collateral the auctioneer is expected to get: /// 0 if liquidator == auctioneer.
// If the vault hasn't been auctioned yet, we calculate what values it'd have if it was started right now
Recommendation:
// If the vault hasn't been auctioned yet, we calculate what values it'd have // if it was started right now.
// Calculate how much collateral to give for paying a certain amount of debt, at a certain time, for a certain vault.
Recommendation:
// Calculate how much collateral to give for paying a certain amount of debt, /// at a certain time, for a certain vault.
#0 - alcueca
2022-07-22T14:19:11Z
Thanks for the typo check.
I'm a 120 character per line maxi.
#1 - ultrasecreth
2022-07-22T14:58:38Z
specialised
is not a typo :)