*: fix exhaustive warnings

helpers.go:50:2 exhaustive missing cases in switch of type dbft.MessageType: dbft.RecoveryRequestType, dbft.RecoveryMessageType internal/consensus/recovery_message.go:39:2 exhaustive missing cases in switch of type dbft.MessageType: dbft.RecoveryRequestType, dbft.RecoveryMessageType Signed-off-by: Roman Khimov <[email protected]>
nspcc-dev · Sep 10, 2024 · aab1cf6 · aab1cf6
1 parent 1fd6aa8
commit aab1cf6
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/helpers.go b/helpers.go
@@ -56,5 +56,8 @@ func (c *cache[H]) addMessage(m ConsensusPayload[H]) {
 		msgs.preCommit[m.ValidatorIndex()] = m
 	case CommitType:
 		msgs.commit[m.ValidatorIndex()] = m
+	default:
+		// Others are recoveries and we don't currently use them.
+		// Theoretically messages could be extracted.
 	}
 }
diff --git a/internal/consensus/recovery_message.go b/internal/consensus/recovery_message.go
@@ -65,6 +65,8 @@ func (m *recoveryMessage) AddPayload(p dbft.ConsensusPayload[crypto.Uint256]) {
 		}
 		copy(cc.Signature[:], p.GetCommit().Signature())
 		m.commitPayloads = append(m.commitPayloads, cc)
+	default:
+		// Other types (recoveries) can't be packed into recovery.
 	}
 }