chore: lint: address feedback from reviews

This commit is contained in:
Rod Vagg 2024-05-13 12:29:13 +10:00
parent 26d3fd2ecc
commit 1882525578
3 changed files with 20 additions and 18 deletions

View File

@ -141,7 +141,10 @@ func createRandomFile(rseed, size int) ([]byte, string, error) {
size = 1600
}
data := make([]byte, size)
_, _ = rand.New(rand.NewSource(int64(rseed))).Read(data)
_, err := rand.New(rand.NewSource(int64(rseed))).Read(data)
if err != nil {
return nil, "", err
}
dir, err := os.MkdirTemp(os.TempDir(), "test-make-deal-")
if err != nil {

View File

@ -201,7 +201,5 @@ func (m *minerAPI) GetUnpaddedCARSize(ctx context.Context, pieceCid cid.Cid) (ui
return 0, xerrors.Errorf("no storage deals found for piece %s", pieceCid)
}
l := pieceInfo.Deals[0].Length
return uint64(l), nil
return uint64(pieceInfo.Deals[0].Length), nil
}

View File

@ -396,19 +396,20 @@ func lookupEthAddress(addr address.Address, st *state.StateTree) (ethtypes.EthAd
return ethtypes.EthAddress{}, err
}
// revive:disable:empty-block easier to grok when the cases are explicit
// Lookup on the target actor and try to get an f410 address.
if actor, err := st.GetActor(idAddr); !errors.Is(err, types.ErrActorNotFound) {
if err != nil {
// Any other error -> fail.
return ethtypes.EthAddress{}, err
}
if actor.Address != nil {
if ethAddr, err := ethtypes.EthAddressFromFilecoinAddress(*actor.Address); err == nil && !ethAddr.IsMaskedID() {
// Conversable into an eth address, use it.
return ethAddr, nil
}
} // else no delegated address -> use masked ID address.
} // else not found -> use a masked ID address
if actor, err := st.GetActor(idAddr); errors.Is(err, types.ErrActorNotFound) {
// Not found -> use a masked ID address
} else if err != nil {
// Any other error -> fail.
return ethtypes.EthAddress{}, err
} else if actor.Address == nil {
// No delegated address -> use masked ID address.
} else if ethAddr, err := ethtypes.EthAddressFromFilecoinAddress(*actor.Address); err == nil && !ethAddr.IsMaskedID() {
// Conversable into an eth address, use it.
return ethAddr, nil
}
return ethtypes.EthAddressFromFilecoinAddress(idAddr)
}
@ -816,8 +817,8 @@ func encodeAsABIHelper(param1 uint64, param2 uint64, data []byte) []byte {
if len(data)%EVM_WORD_SIZE != 0 {
totalWords++
}
l := totalWords * EVM_WORD_SIZE
buf := make([]byte, l)
sz := totalWords * EVM_WORD_SIZE
buf := make([]byte, sz)
offset := 0
// Below, we use copy instead of "appending" to preserve all the zero padding.
for _, arg := range staticArgs {