proto: change type + a cleanup (#5107)

- drop Height & Base from StatusRequest
It does not make sense nor it's used anywhere currently. Also, there
seem to be no trace of these fields in the ADR-40 (blockchain reactor
v2).

- change PacketMsg#EOF type from int32 to bool
This commit is contained in:
Anton Kaliaev
2020-07-13 14:24:17 +04:00
committed by GitHub
parent 37545bab88
commit 730e16566e
13 changed files with 91 additions and 194 deletions

View File

@@ -410,10 +410,7 @@ FOR_LOOP:
// BroadcastStatusRequest broadcasts `BlockStore` base and height.
func (bcR *BlockchainReactor) BroadcastStatusRequest() error {
bm, err := bc.EncodeMsg(&bcproto.StatusRequest{
Base: bcR.store.Base(),
Height: bcR.store.Height(),
})
bm, err := bc.EncodeMsg(&bcproto.StatusRequest{})
if err != nil {
bcR.Logger.Error("could not convert msg to proto", "err", err)
return fmt.Errorf("could not convert msg to proto: %w", err)