fix: fixing review comments

committee/committee.go

@@ -69,8 +69,6 @@ func (c *committee) Update(lastRound int16, joined []*validator.Validator) {
 		} else {
 			committeeVal.UpdateLastSortitionHeight(val.LastSortitionHeight())
 
-			// TODO: PRoofread the comments here
-			//
 			// Ensure that a validator's stake and bonding properties
 			// remain unchanged while they are part of the committee.
 			// Refer to the Bond executor for additional details.

execution/executor/sortition.go

@@ -44,7 +44,7 @@ func (e *SortitionExecutor) Execute(trx *tx.Tx, sb sandbox.Sandbox) error {
 			return errors.Errorf(errors.ErrInvalidSequence,
 				"expected: %v, got: %v", val.Sequence()+1, trx.Sequence())
 		}
-		// Check for the duplicated sortition transactions
+		// Check for the duplicated or expired sortition transactions
 		if sortitionHeight <= val.LastSortitionHeight() {
 			return errors.Errorf(errors.ErrInvalidTx,
 				"duplicated sortition transaction")

types/account/account.go

@@ -70,13 +70,11 @@ func (acc *Account) AddToBalance(amt int64) {
 	acc.data.Balance += amt
 }
 
-// IncSequence increments the account's sequence every time it signs a transaction.
 // IncSequence increases the sequence anytime this account signs a transaction.
 func (acc *Account) IncSequence() {
 	acc.data.Sequence++
 }
 
-// Hash returns the hash of this account.
 // Hash calculates and returns the hash of the account.
 func (acc *Account) Hash() hash.Hash {
 	bs, err := acc.Bytes()

www/http/blockchain.go

@@ -26,15 +26,7 @@ func (s *Server) BlockchainHandler(w http.ResponseWriter, _ *http.Request) {
 	tm.addRowAmount("Committee Power", res.CommitteePower)
 	for i, val := range res.CommitteeValidators {
 		tm.addRowInt("--- Validator", i+1)
-		tm.addRowString("Public Key", val.PublicKey)
-		tm.addRowValAddress("Address", val.Address)
-		tm.addRowInt("Number", int(val.Number))
-		tm.addRowInt("Sequence", int(val.Sequence))
-		tm.addRowAmount("Stake", val.Stake)
-		tm.addRowInt("LastBondingHeight", int(val.LastBondingHeight))
-		tm.addRowInt("LastSortitionHeight", int(val.LastSortitionHeight))
-		tm.addRowInt("UnbondingHeight", int(val.UnbondingHeight))
-		tm.addRowBytes("Hash", val.Hash)
+		s.writeValidatorTable(w, val)
 	}
 
 	s.writeHTML(w, tm.html())

www/http/validator.go

@@ -18,19 +18,7 @@ func (s *Server) GetValidatorHandler(w http.ResponseWriter, r *http.Request) {
 		return
 	}
 
-	val := res.Validator
-	tm := newTableMaker()
-	tm.addRowString("Public Key", val.PublicKey)
-	tm.addRowValAddress("Address", val.Address)
-	tm.addRowInt("Number", int(val.Number))
-	tm.addRowInt("Sequence", int(val.Sequence))
-	tm.addRowAmount("Stake", val.Stake)
-	tm.addRowInt("LastBondingHeight", int(val.LastBondingHeight))
-	tm.addRowInt("LastSortitionHeight", int(val.LastSortitionHeight))
-	tm.addRowInt("UnbondingHeight", int(val.UnbondingHeight))
-	tm.addRowBytes("Hash", val.Hash)
-
-	s.writeHTML(w, tm.html())
+	s.writeValidatorTable(w, res.Validator)
 }
 
 // GetValidatorByNumberHandler returns a handler to get validator by number.
@@ -51,7 +39,10 @@ func (s *Server) GetValidatorByNumberHandler(w http.ResponseWriter, r *http.Requ
 		return
 	}
 
-	val := res.Validator
+	s.writeValidatorTable(w, res.Validator)
+}
+
+func (s *Server) writeValidatorTable(w http.ResponseWriter, val *pactus.ValidatorInfo) {
 	tm := newTableMaker()
 	tm.addRowString("Public Key", val.PublicKey)
 	tm.addRowValAddress("Address", val.Address)