Submitted by pauliax
function withdrawAll in BaseStrategy declares returns (uint256 balance), however, no actual value is returned. function reinvest in MyStrategy declares to return uint256 reinvested, however, it also actually does not return anything so they always get assigned a default value of 0.
Either remove the return declarations or return the intended values. Otherwise, it may confuse other protocols that later may want to integrate with you.
GalloDaSballo (BadgerDAO) confirmed:
