QA Report #378
Labels
bug
Something isn't working
QA (Quality Assurance)
Assets are not at risk. State handling, function incorrect as to spec, issues with clarity, syntax
Typos
marketplace/MarketPlace.sol: L11
/// @notice This contract is in charge of managing the avaialable principals for each loan market.
Change
avaialable
toavaialable
marketplace/MarketPlace.sol: L34
/// @notice The principal tokens those addresses represent correspond to their Principals enum value, thus the array should be ordered in that way
Change
those addresses represent
towhose addresses
marketplace/MarketPlace.sol: L50
/// @notice intializes the MarketPlace contract
Change
intializes
toinitializes
lender/Lender.sol: L6
Change
market place
tomarketplace
lender/Lender.sol: L83
// max is the maximum integer value for a 256 unsighed integer
Change
unsighed
tounsigned
The same typo (
gauruntee
) occurs in both lines referenced below:lender/Lender.sol: L500
lender/Lender.sol: L504
Example:
Change
gauruntee
toguarantee
in both caseslender/Lender.sol: L650
// send the remaing amount to the given yield pool
Change
remaing
toremaining
lender/Lender.sol: L679
/// @return uint256 The total for for the given amount
Remove the repeated word
for
The same typo (
prinicipal
) occurs in both lines referenced below:redeemer/Redeemer.sol: L125
redeemer/Redeemer.sol: L189
Example:
// Burn the prinicipal token from Illuminate
Change
prinicipal
toprincipal
in both casesredeemer/Redeemer.sol: L237
/// @param d Sense contract that splits the loan's prinicpal and yield
Change
prinicpal
toprincipal
redeemer/Redeemer.sol: L273
/// @param a amount of the underlying asset to be burned and sent to the to
Replace
to the to
withto the t address
Comments referring to unfinished work or open items
Such comments should be addressed and modified or removed
lender/Lender.sol: L561
// Potentially add redundant implied maturity calculation
The text was updated successfully, but these errors were encountered: