message: [consensus][minor] Minor comment fixes for LedgerInfo
  Closes:
  Approved by: dmitri-perelman
diff: @@ -32,20 +32,20 @@ use std::{
  ///
  /// Second, the structure contains a `consensus_data_hash` value. This is the hash of an internal
  /// data structure that represents a block that is voted on in HotStuff. If 2f+1 signatures are
  -/// gathered on the same ledger info that represents a Quorum Certificate (QC) on the HotStuff
  +/// gathered on the same ledger info that represents a Quorum Certificate (QC) on the consensus
  /// data.
  ///
  -/// Combining these two concepts when the consensus algorithm votes on a block B it votes for a
  +/// Combining these two concepts, when a validator votes on a block, B it votes for a
  /// LedgerInfo with the `version` being the latest version that will be committed if B gets 2f+1
  /// votes. It sets `consensus_data_hash` to represent B so that if those 2f+1 votes are gathered a
  /// QC is formed on B.
  #[derive(Clone, Debug, Eq, PartialEq, Serialize, Deserialize)]
  #[cfg_attr(any(test, feature = "testing"), derive(Arbitrary))]
  pub struct LedgerInfo {
  - /// The version of latest transaction in the ledger.
  + /// The version of the latest transaction in the ledger.
  version: Version,
  - /// The root hash of transaction accumulator.
  + /// The root hash of transaction accumulator that includes the latest transaction.
  transaction_accumulator_hash: HashValue,
  /// Hash of consensus specific data that is opaque to all parts of the system other than
  @@ -59,11 +59,11 @@ pub struct LedgerInfo {
  /// Epoch number corresponds to the set of validators that are active for this ledger info.
  epoch_num: u64,
  - // Timestamp that represents the microseconds since the epoch (unix time) that is
  - // generated by the proposer of the block. This is strictly increasing with every block.
  - // If a client reads a timestamp > the one they specified for transaction expiration time,
  - // they can be certain that their transaction will never be included in a block in the future
  - // (assuming that their transaction has not yet been included)
  + /// Timestamp that represents the microseconds since the epoch (unix time) that is
  + /// generated by the proposer of the block. This is strictly increasing with every block.
  + /// If a client reads a timestamp > the one they specified for transaction expiration time,
  + /// they can be certain that their transaction will never be included in a block in the future
  + /// (assuming that their transaction has not yet been included)
  timestamp_usecs: u64,
  /// An optional field keeping the set of new validators to start the next epoch.
  @@ -87,8 +87,8 @@ impl Display for LedgerInfo {
  }
  impl LedgerInfo {
  - /// Constructs a `LedgerInfo` object at a specific version using given transaction accumulator
  - /// root and hot stuff data hash.
  + /// Constructs a `LedgerInfo` object at a specific version using a given
  + /// transaction accumulator root and consensus data hash.
  pub fn new(
  version: Version,
  transaction_accumulator_hash: HashValue,
  @@ -224,11 +224,11 @@ impl CryptoHash for LedgerInfo {
  }
  }
  -// The validator node returns this structure which includes signatures
  -// from each validator to confirm the state. The client needs to only pass back
  -// the LedgerInfo element since the validator node doesn't need to know the signatures
  -// again when the client performs a query, those are only there for the client
  -// to be able to verify the state
  +/// The validator node returns this structure which includes signatures
  +/// from validators that confirm the state. The client needs to only pass back
  +/// the LedgerInfo element since the validator node doesn't need to know the signatures
  +/// again when the client performs a query, those are only there for the client
  +/// to be able to verify the state
  #[derive(Clone, Debug, Eq, PartialEq, Serialize, Deserialize)]
  pub struct LedgerInfoWithSignatures<Sig> {
  ledger_info: LedgerInfo,
  