ALPACA Token

FairLaunch.safeAlpacaTransfer(address,uint256) (#1588-1595) ignores return value by alpaca.transfer(_to,alpacaBal) (#1591)
FairLaunch.safeAlpacaTransfer(address,uint256) (#1588-1595) ignores return value by alpaca.transfer(_to,_amount) (#1593)
Use SafeERC20, or ensure that the transfer/transferFrom return value is checked.

Additional information: link

FairLaunch.updatePool(uint256) (#1487-1513) uses a dangerous strict equality:
- lpSupply == 0 (#1493)
AlpacaToken._writeCheckpoint(address,uint32,uint256,uint256) (#1220-1236) uses a dangerous strict equality:
- nCheckpoints > 0 && checkpoints[delegatee][nCheckpoints - 1].fromBlock == blockNumber (#1228)
FairLaunch.isDuplicatedPool(address) (#1436-1442) uses a dangerous strict equality:
- poolInfo[_pid].stakeToken == _stakeToken (#1439)
Don't use strict equality to determine if an account has enough Ether or tokens.

Additional information: link

AlpacaToken.delegateBySig(address,uint256,uint256,uint8,bytes32,bytes32) (#1112-1133) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(now <= expiry,ALPACA::delegateBySig: signature expired) (#1131)
Avoid relying on block.timestamp.

Additional information: link

solc-0.6.6 is not recommended for deployment
Pragma version0.6.6 (#5) allows old versions
Deploy with any of the following Solidity versions: 0.5.16 - 0.5.17, 0.6.11 - 0.6.12, 0.7.5 - 0.7.6 Use a simple pragma version that allows any of these versions. Consider using the latest version of Solidity for testing.

Additional information: link

Low level call in Address.functionCallWithValue(address,bytes,uint256,string) (#396-403):
- (success,returndata) = target.call{value: value}(data) (#401)
Low level call in Address.functionDelegateCall(address,bytes,string) (#445-451):
- (success,returndata) = target.delegatecall(data) (#449)
Low level call in Address.sendValue(address,uint256) (#335-341):
- (success) = recipient.call{value: amount}() (#339)
Low level call in Address.functionStaticCall(address,bytes,string) (#421-427):
- (success,returndata) = target.staticcall(data) (#425)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence

Additional information: link

FairLaunch.pendingAlpaca(uint256,address) (#1465-1476) performs a multiplication on the result of a division:
-alpacaReward = multiplier.mul(alpacaPerBlock).mul(pool.allocPoint).div(totalAllocPoint) (#1472)
-accAlpacaPerShare = accAlpacaPerShare.add(alpacaReward.mul(1e12).div(lpSupply)) (#1473)
FairLaunch.updatePool(uint256) (#1487-1513) performs a multiplication on the result of a division:
-alpacaReward = multiplier.mul(alpacaPerBlock).mul(pool.allocPoint).div(totalAllocPoint) (#1498)
-pool.accAlpacaPerShare = pool.accAlpacaPerShare.add(alpacaReward.mul(1e12).div(lpSupply)) (#1501)
FairLaunch.updatePool(uint256) (#1487-1513) performs a multiplication on the result of a division:
-alpacaBonusPortion = bonusEndBlock.sub(pool.lastRewardBlock).mul(bonusMultiplier).mul(alpacaPerBlock).mul(pool.allocPoint).div(totalAllocPoint) (#1508)
-pool.accAlpacaPerShareTilBonusEnd = pool.accAlpacaPerShareTilBonusEnd.add(alpacaBonusPortion.mul(1e12).div(lpSupply)) (#1510)
FairLaunch.updatePool(uint256) (#1487-1513) performs a multiplication on the result of a division:
-alpaca.lock(devaddr,alpacaBonusPortion.div(10).mul(bonusLockUpBps).div(10000)) (#1509)
FairLaunch.updatePool(uint256) (#1487-1513) performs a multiplication on the result of a division:
-alpaca.lock(devaddr,alpacaReward.div(10).mul(bonusLockUpBps).div(10000)) (#1504)
Consider ordering multiplication before division.

Additional information: link

lockOf(address) should be declared external:
- AlpacaToken.lockOf(address) (#975-977)
addPool(uint256,address,bool) should be declared external:
- FairLaunch.addPool(uint256,address,bool) (#1391-1412)
unlock() should be declared external:
- AlpacaToken.unlock() (#1018-1027)
transferFrom(address,address,uint256) should be declared external:
- ERC20.transferFrom(address,address,uint256) (#757-761)
emergencyWithdraw(uint256) should be declared external:
- FairLaunch.emergencyWithdraw(uint256) (#1578-1585)
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#598-601)
manualMint(address,uint256) should be declared external:
- FairLaunch.manualMint(address,uint256) (#1448-1450)
setDev(address) should be declared external:
- FairLaunch.setDev(address) (#1367-1370)
withdrawAll(address,uint256) should be declared external:
- FairLaunch.withdrawAll(address,uint256) (#1536-1538)
transferAll(address) should be declared external:
- AlpacaToken.transferAll(address) (#1030-1045)
lastUnlockBlock(address) should be declared external:
- AlpacaToken.lastUnlockBlock(address) (#979-981)
decimals() should be declared external:
- ERC20.decimals() (#694-696)
setBonus(uint256,uint256,uint256) should be declared external:
- FairLaunch.setBonus(uint256,uint256,uint256) (#1378-1388)
decreaseAllowance(address,uint256) should be declared external:
- ERC20.decreaseAllowance(address,uint256) (#794-797)
withdraw(address,uint256,uint256) should be declared external:
- FairLaunch.withdraw(address,uint256,uint256) (#1532-1534)
symbol() should be declared external:
- ERC20.symbol() (#677-679)
transfer(address,uint256) should be declared external:
- ERC20.transfer(address,uint256) (#720-723)
lock(address,uint256) should be declared external:
- AlpacaToken.lock(address,uint256) (#983-997)
increaseAllowance(address,uint256) should be declared external:
- ERC20.increaseAllowance(address,uint256) (#775-778)
burn(address,uint256) should be declared external:
- AlpacaToken.burn(address,uint256) (#967-969)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#607-611)
setPool(uint256,uint256,bool) should be declared external:
- FairLaunch.setPool(uint256,uint256,bool) (#1415-1425)
setAlpacaPerBlock(uint256) should be declared external:
- FairLaunch.setAlpacaPerBlock(uint256) (#1372-1374)
setReleaseBlock(uint256,uint256) should be declared external:
- AlpacaToken.setReleaseBlock(uint256,uint256) (#938-942)
totalBalanceOf(address) should be declared external:
- AlpacaToken.totalBalanceOf(address) (#971-973)
deposit(address,uint256,uint256) should be declared external:
- FairLaunch.deposit(address,uint256,uint256) (#1516-1529)
approve(address,uint256) should be declared external:
- ERC20.approve(address,uint256) (#739-742)
unlockedSupply() should be declared external:
- AlpacaToken.unlockedSupply() (#948-950)
harvest(uint256) should be declared external:
- FairLaunch.harvest(uint256) (#1557-1564)
allowance(address,address) should be declared external:
- ERC20.allowance(address,address) (#728-730)
Use the external attribute for functions never called from the contract.

Additional information: link

FairLaunch.updatePool(uint256) (#1487-1513) has external calls inside a loop: lpSupply = IERC20(pool.stakeToken).balanceOf(address(this)) (#1492)
FairLaunch.updatePool(uint256) (#1487-1513) has external calls inside a loop: alpaca.lock(devaddr,alpacaReward.div(10).mul(bonusLockUpBps).div(10000)) (#1504)
FairLaunch.updatePool(uint256) (#1487-1513) has external calls inside a loop: alpaca.mint(devaddr,alpacaReward.div(10)) (#1499)
FairLaunch.updatePool(uint256) (#1487-1513) has external calls inside a loop: alpaca.mint(address(this),alpacaReward) (#1500)
FairLaunch.updatePool(uint256) (#1487-1513) has external calls inside a loop: alpaca.lock(devaddr,alpacaBonusPortion.div(10).mul(bonusLockUpBps).div(10000)) (#1509)
Favor pull over push strategy for external calls.

Additional information: link

FairLaunch.setDev(address)._devaddr (#1367) lacks a zero-check on :
- devaddr = _devaddr (#1369)
FairLaunch.constructor(AlpacaToken,address,uint256,uint256,uint256,uint256)._devaddr (#1341) lacks a zero-check on :
- devaddr = _devaddr (#1350)
Check that the address is not zero.

Additional information: link

Reentrancy in FairLaunch.updatePool(uint256) (#1487-1513):
External calls:
- alpaca.mint(devaddr,alpacaReward.div(10)) (#1499)
- alpaca.mint(address(this),alpacaReward) (#1500)
State variables written after the call(s):
- pool.accAlpacaPerShare = pool.accAlpacaPerShare.add(alpacaReward.mul(1e12).div(lpSupply)) (#1501)
Reentrancy in FairLaunch.updatePool(uint256) (#1487-1513):
External calls:
- alpaca.mint(devaddr,alpacaReward.div(10)) (#1499)
- alpaca.mint(address(this),alpacaReward) (#1500)
- alpaca.lock(devaddr,alpacaReward.div(10).mul(bonusLockUpBps).div(10000)) (#1504)
State variables written after the call(s):
- pool.accAlpacaPerShareTilBonusEnd = pool.accAlpacaPerShare (#1505)
Reentrancy in FairLaunch.setPool(uint256,uint256,bool) (#1415-1425):
External calls:
- massUpdatePools() (#1421)
- alpaca.mint(devaddr,alpacaReward.div(10)) (#1499)
- alpaca.mint(address(this),alpacaReward) (#1500)
- alpaca.lock(devaddr,alpacaReward.div(10).mul(bonusLockUpBps).div(10000)) (#1504)
- alpaca.lock(devaddr,alpacaBonusPortion.div(10).mul(bonusLockUpBps).div(10000)) (#1509)
State variables written after the call(s):
- poolInfo[_pid].allocPoint = _allocPoint (#1424)
- totalAllocPoint = totalAllocPoint.sub(poolInfo[_pid].allocPoint).add(_allocPoint) (#1423)
Reentrancy in FairLaunch._withdraw(address,uint256,uint256) (#1540-1554):
External calls:
- updatePool(_pid) (#1545)
- alpaca.mint(devaddr,alpacaReward.div(10)) (#1499)
- alpaca.mint(address(this),alpacaReward) (#1500)
- alpaca.lock(devaddr,alpacaReward.div(10).mul(bonusLockUpBps).div(10000)) (#1504)
- alpaca.lock(devaddr,alpacaBonusPortion.div(10).mul(bonusLockUpBps).div(10000)) (#1509)
- _harvest(_for,_pid) (#1546)
- alpaca.transfer(_to,alpacaBal) (#1591)
- alpaca.transfer(_to,_amount) (#1593)
- alpaca.lock(_to,bonus.mul(bonusLockUpBps).div(10000)) (#1574)
State variables written after the call(s):
- user.amount = user.amount.sub(_amount) (#1547)
- user.rewardDebt = user.amount.mul(pool.accAlpacaPerShare).div(1e12) (#1548)
- user.bonusDebt = user.amount.mul(pool.accAlpacaPerShareTilBonusEnd).div(1e12) (#1549)
Reentrancy in FairLaunch.harvest(uint256) (#1557-1564):
External calls:
- updatePool(_pid) (#1560)
- alpaca.mint(devaddr,alpacaReward.div(10)) (#1499)
- alpaca.mint(address(this),alpacaReward) (#1500)
- alpaca.lock(devaddr,alpacaReward.div(10).mul(bonusLockUpBps).div(10000)) (#1504)
- alpaca.lock(devaddr,alpacaBonusPortion.div(10).mul(bonusLockUpBps).div(10000)) (#1509)
- _harvest(msg.sender,_pid) (#1561)
- alpaca.transfer(_to,alpacaBal) (#1591)
- alpaca.transfer(_to,_amount) (#1593)
- alpaca.lock(_to,bonus.mul(bonusLockUpBps).div(10000)) (#1574)
State variables written after the call(s):
- user.rewardDebt = user.amount.mul(pool.accAlpacaPerShare).div(1e12) (#1562)
- user.bonusDebt = user.amount.mul(pool.accAlpacaPerShareTilBonusEnd).div(1e12) (#1563)
Reentrancy in FairLaunch.deposit(address,uint256,uint256) (#1516-1529):
External calls:
- updatePool(_pid) (#1521)
- alpaca.mint(devaddr,alpacaReward.div(10)) (#1499)
- alpaca.mint(address(this),alpacaReward) (#1500)
- alpaca.lock(devaddr,alpacaReward.div(10).mul(bonusLockUpBps).div(10000)) (#1504)
- alpaca.lock(devaddr,alpacaBonusPortion.div(10).mul(bonusLockUpBps).div(10000)) (#1509)
- _harvest(_for,_pid) (#1522)
- alpaca.transfer(_to,alpacaBal) (#1591)
- alpaca.transfer(_to,_amount) (#1593)
- alpaca.lock(_to,bonus.mul(bonusLockUpBps).div(10000)) (#1574)
State variables written after the call(s):
- user.fundedBy = msg.sender (#1523)
Reentrancy in FairLaunch.emergencyWithdraw(uint256) (#1578-1585):
External calls:
- IERC20(pool.stakeToken).safeTransfer(address(msg.sender),user.amount) (#1581)
State variables written after the call(s):
- user.amount = 0 (#1583)
- user.rewardDebt = 0 (#1584)
Reentrancy in FairLaunch.updatePool(uint256) (#1487-1513):
External calls:
- alpaca.mint(devaddr,alpacaReward.div(10)) (#1499)
- alpaca.mint(address(this),alpacaReward) (#1500)
- alpaca.lock(devaddr,alpacaReward.div(10).mul(bonusLockUpBps).div(10000)) (#1504)
- alpaca.lock(devaddr,alpacaBonusPortion.div(10).mul(bonusLockUpBps).div(10000)) (#1509)
State variables written after the call(s):
- pool.accAlpacaPerShareTilBonusEnd = pool.accAlpacaPerShareTilBonusEnd.add(alpacaBonusPortion.mul(1e12).div(lpSupply)) (#1510)
- pool.lastRewardBlock = block.number (#1512)
Reentrancy in FairLaunch.addPool(uint256,address,bool) (#1391-1412):
External calls:
- massUpdatePools() (#1397)
- alpaca.mint(devaddr,alpacaReward.div(10)) (#1499)
- alpaca.mint(address(this),alpacaReward) (#1500)
- alpaca.lock(devaddr,alpacaReward.div(10).mul(bonusLockUpBps).div(10000)) (#1504)
- alpaca.lock(devaddr,alpacaBonusPortion.div(10).mul(bonusLockUpBps).div(10000)) (#1509)
State variables written after the call(s):
- poolInfo.push(PoolInfo(_stakeToken,_allocPoint,lastRewardBlock,0,0)) (#1403-1411)
- totalAllocPoint = totalAllocPoint.add(_allocPoint) (#1402)
Reentrancy in FairLaunch.deposit(address,uint256,uint256) (#1516-1529):
External calls:
- updatePool(_pid) (#1521)
- alpaca.mint(devaddr,alpacaReward.div(10)) (#1499)
- alpaca.mint(address(this),alpacaReward) (#1500)
- alpaca.lock(devaddr,alpacaReward.div(10).mul(bonusLockUpBps).div(10000)) (#1504)
- alpaca.lock(devaddr,alpacaBonusPortion.div(10).mul(bonusLockUpBps).div(10000)) (#1509)
- _harvest(_for,_pid) (#1522)
- alpaca.transfer(_to,alpacaBal) (#1591)
- alpaca.transfer(_to,_amount) (#1593)
- alpaca.lock(_to,bonus.mul(bonusLockUpBps).div(10000)) (#1574)
- IERC20(pool.stakeToken).safeTransferFrom(address(msg.sender),address(this),_amount) (#1524)
State variables written after the call(s):
- user.amount = user.amount.add(_amount) (#1525)
- user.rewardDebt = user.amount.mul(pool.accAlpacaPerShare).div(1e12) (#1526)
- user.bonusDebt = user.amount.mul(pool.accAlpacaPerShareTilBonusEnd).div(1e12) (#1527)
Apply the check-effects-interactions pattern.

Additional information: link

AlpacaToken.setReleaseBlock(uint256,uint256) (#938-942) should emit an event for:
- startReleaseBlock = _startReleaseBlock (#940)
- endReleaseBlock = _endReleaseBlock (#941)
FairLaunch.setAlpacaPerBlock(uint256) (#1372-1374) should emit an event for:
- alpacaPerBlock = _alpacaPerBlock (#1373)
FairLaunch.addPool(uint256,address,bool) (#1391-1412) should emit an event for:
- totalAllocPoint = totalAllocPoint.add(_allocPoint) (#1402)
FairLaunch.setBonus(uint256,uint256,uint256) (#1378-1388) should emit an event for:
- bonusMultiplier = _bonusMultiplier (#1385)
- bonusEndBlock = _bonusEndBlock (#1386)
- bonusLockUpBps = _bonusLockUpBps (#1387)
FairLaunch.setPool(uint256,uint256,bool) (#1415-1425) should emit an event for:
- totalAllocPoint = totalAllocPoint.sub(poolInfo[_pid].allocPoint).add(_allocPoint) (#1423)
Emit an event for critical parameter changes.

Additional information: link

Reentrancy in FairLaunch.deposit(address,uint256,uint256) (#1516-1529):
External calls:
- updatePool(_pid) (#1521)
- alpaca.mint(devaddr,alpacaReward.div(10)) (#1499)
- alpaca.mint(address(this),alpacaReward) (#1500)
- alpaca.lock(devaddr,alpacaReward.div(10).mul(bonusLockUpBps).div(10000)) (#1504)
- alpaca.lock(devaddr,alpacaBonusPortion.div(10).mul(bonusLockUpBps).div(10000)) (#1509)
- _harvest(_for,_pid) (#1522)
- alpaca.transfer(_to,alpacaBal) (#1591)
- alpaca.transfer(_to,_amount) (#1593)
- alpaca.lock(_to,bonus.mul(bonusLockUpBps).div(10000)) (#1574)
- IERC20(pool.stakeToken).safeTransferFrom(address(msg.sender),address(this),_amount) (#1524)
Event emitted after the call(s):
- Deposit(msg.sender,_pid,_amount) (#1528)
Reentrancy in FairLaunch.emergencyWithdraw(uint256) (#1578-1585):
External calls:
- IERC20(pool.stakeToken).safeTransfer(address(msg.sender),user.amount) (#1581)
Event emitted after the call(s):
- EmergencyWithdraw(msg.sender,_pid,user.amount) (#1582)
Reentrancy in FairLaunch._withdraw(address,uint256,uint256) (#1540-1554):
External calls:
- updatePool(_pid) (#1545)
- alpaca.mint(devaddr,alpacaReward.div(10)) (#1499)
- alpaca.mint(address(this),alpacaReward) (#1500)
- alpaca.lock(devaddr,alpacaReward.div(10).mul(bonusLockUpBps).div(10000)) (#1504)
- alpaca.lock(devaddr,alpacaBonusPortion.div(10).mul(bonusLockUpBps).div(10000)) (#1509)
- _harvest(_for,_pid) (#1546)
- alpaca.transfer(_to,alpacaBal) (#1591)
- alpaca.transfer(_to,_amount) (#1593)
- alpaca.lock(_to,bonus.mul(bonusLockUpBps).div(10000)) (#1574)
- IERC20(pool.stakeToken).safeTransfer(address(msg.sender),_amount) (#1551)
Event emitted after the call(s):
- Withdraw(msg.sender,_pid,user.amount) (#1553)
Apply the check-effects-interactions pattern.

Additional information: link

AlpacaToken.getChainId() (#1243-1249) uses assembly
- INLINE ASM (#1245-1247)
Address.isContract(address) (#308-317) uses assembly
- INLINE ASM (#315)
Address._verifyCallResult(bool,bytes,string) (#453-470) uses assembly
- INLINE ASM (#462-465)
Do not use evm assembly.

Additional information: link

SafeMath.tryDiv(uint256,uint256) (#134-137) is never used and should be removed
Address.sendValue(address,uint256) (#335-341) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#386-388) is never used and should be removed
SafeMath.tryMod(uint256,uint256) (#144-147) is never used and should be removed
Address.functionDelegateCall(address,bytes,string) (#445-451) is never used and should be removed
Address.functionDelegateCall(address,bytes) (#435-437) is never used and should be removed
SafeERC20.safeIncreaseAllowance(IERC20,address,uint256) (#512-515) is never used and should be removed
SafeERC20.safeApprove(IERC20,address,uint256) (#501-510) is never used and should be removed
SafeMath.tryAdd(uint256,uint256) (#98-102) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#284-287) is never used and should be removed
SafeMath.div(uint256,uint256,string) (#264-267) is never used and should be removed
Context._msgData() (#556-559) is never used and should be removed
Address.functionStaticCall(address,bytes) (#411-413) is never used and should be removed
SafeMath.mod(uint256,uint256) (#226-229) is never used and should be removed
SafeMath.tryMul(uint256,uint256) (#119-127) is never used and should be removed
SafeMath.trySub(uint256,uint256) (#109-112) is never used and should be removed
SafeERC20.safeDecreaseAllowance(IERC20,address,uint256) (#517-520) is never used and should be removed
Address.functionStaticCall(address,bytes,string) (#421-427) is never used and should be removed
Address.functionCall(address,bytes) (#361-363) is never used and should be removed
Remove unused functions.

Additional information: link

Parameter FairLaunch.setPool(uint256,uint256,bool)._allocPoint (#1417) is not in mixedCase
Parameter FairLaunch.deposit(address,uint256,uint256)._for (#1516) is not in mixedCase
Parameter FairLaunch.withdraw(address,uint256,uint256)._amount (#1532) is not in mixedCase
Parameter AlpacaToken.burn(address,uint256)._amount (#967) is not in mixedCase
Parameter FairLaunch.withdraw(address,uint256,uint256)._for (#1532) is not in mixedCase
Parameter FairLaunch.setPool(uint256,uint256,bool)._pid (#1416) is not in mixedCase
Parameter FairLaunch.deposit(address,uint256,uint256)._pid (#1516) is not in mixedCase
Parameter FairLaunch.withdrawAll(address,uint256)._for (#1536) is not in mixedCase
Parameter FairLaunch.setBonus(uint256,uint256,uint256)._bonusMultiplier (#1379) is not in mixedCase
Parameter AlpacaToken.canUnlockAmount(address)._account (#999) is not in mixedCase
Parameter AlpacaToken.mint(address,uint256)._to (#961) is not in mixedCase
Parameter FairLaunch.emergencyWithdraw(uint256)._pid (#1578) is not in mixedCase
Parameter FairLaunch.safeAlpacaTransfer(address,uint256)._to (#1588) is not in mixedCase
Parameter FairLaunch.pendingAlpaca(uint256,address)._user (#1465) is not in mixedCase
Parameter FairLaunch.setDev(address)._devaddr (#1367) is not in mixedCase
Parameter FairLaunch.deposit(address,uint256,uint256)._amount (#1516) is not in mixedCase
Parameter AlpacaToken.lockOf(address)._account (#975) is not in mixedCase
Parameter AlpacaToken.burn(address,uint256)._account (#967) is not in mixedCase
Parameter FairLaunch.withdrawAll(address,uint256)._pid (#1536) is not in mixedCase
Parameter FairLaunch.addPool(uint256,address,bool)._withUpdate (#1394) is not in mixedCase
Parameter FairLaunch.getMultiplier(uint256,uint256)._currentBlock (#1453) is not in mixedCase
Parameter FairLaunch.manualMint(address,uint256)._amount (#1448) is not in mixedCase
Parameter FairLaunch.addPool(uint256,address,bool)._allocPoint (#1392) is not in mixedCase
Parameter FairLaunch.setBonus(uint256,uint256,uint256)._bonusEndBlock (#1380) is not in mixedCase
Parameter AlpacaToken.manualMint(address,uint256)._to (#956) is not in mixedCase
Parameter AlpacaToken.lastUnlockBlock(address)._account (#979) is not in mixedCase
Parameter AlpacaToken.totalBalanceOf(address)._account (#971) is not in mixedCase
Parameter AlpacaToken.setReleaseBlock(uint256,uint256)._endReleaseBlock (#938) is not in mixedCase
Parameter FairLaunch.safeAlpacaTransfer(address,uint256)._amount (#1588) is not in mixedCase
Parameter AlpacaToken.lock(address,uint256)._account (#983) is not in mixedCase
Parameter FairLaunch.updatePool(uint256)._pid (#1487) is not in mixedCase
Parameter FairLaunch.setAlpacaPerBlock(uint256)._alpacaPerBlock (#1372) is not in mixedCase
Parameter AlpacaToken.manualMint(address,uint256)._amount (#956) is not in mixedCase
Parameter AlpacaToken.transferAll(address)._to (#1030) is not in mixedCase
Parameter FairLaunch.manualMint(address,uint256)._to (#1448) is not in mixedCase
Parameter FairLaunch.pendingAlpaca(uint256,address)._pid (#1465) is not in mixedCase
Parameter FairLaunch.setBonus(uint256,uint256,uint256)._bonusLockUpBps (#1381) is not in mixedCase
Parameter AlpacaToken.mint(address,uint256)._amount (#961) is not in mixedCase
Parameter FairLaunch.getMultiplier(uint256,uint256)._lastRewardBlock (#1453) is not in mixedCase
Parameter FairLaunch.harvest(uint256)._pid (#1557) is not in mixedCase
Parameter AlpacaToken.lock(address,uint256)._amount (#983) is not in mixedCase
Parameter FairLaunch.addPool(uint256,address,bool)._stakeToken (#1393) is not in mixedCase
Parameter FairLaunch.withdraw(address,uint256,uint256)._pid (#1532) is not in mixedCase
Variable AlpacaToken._delegates (#1054) is not in mixedCase
Parameter FairLaunch.isDuplicatedPool(address)._stakeToken (#1436) is not in mixedCase
Parameter FairLaunch.setPool(uint256,uint256,bool)._withUpdate (#1418) is not in mixedCase
Parameter AlpacaToken.setReleaseBlock(uint256,uint256)._startReleaseBlock (#938) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

Redundant expression "this (#557)" inContext (#551-560)
Remove redundant statements if they congest code but offer no value.

Additional information: link

AlpacaToken.slitherConstructorVariables() (#914-1250) uses literals with too many digits:
- manualMintLimit = 8000000e18 (#920)
AlpacaToken.slitherConstructorVariables() (#914-1250) uses literals with too many digits:
- _cap = 188000000e18 (#915)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

AlpacaToken.manualMinted (#921) should be constant
AlpacaToken.manualMintLimit (#920) should be constant
AlpacaToken._cap (#915) should be constant
Add the constant attributes to state variables that never change.

Additional information: link