message: doc: Clarify that vpindexToConnect is in reverse order
  Also, style-fixups of touched code
diff: @@ -1172,16 +1172,14 @@ static void CheckForkWarningConditions() EXCLUSIVE_LOCKS_REQUIRED(cs_main)
  AssertLockHeld(cs_main);
  // Before we get past initial download, we cannot reliably alert about forks
  // (we assume we don't get stuck on a fork before finishing our initial sync)
  - if (::ChainstateActive().IsInitialBlockDownload())
  + if (::ChainstateActive().IsInitialBlockDownload()) {
  return;
  + }
  - if (pindexBestInvalid && pindexBestInvalid->nChainTrust > ::ChainActive().Tip()->nChainTrust + (GetBlockProof(*::ChainActive().Tip()) * 6))
  - {
  + if (pindexBestInvalid && pindexBestInvalid->nChainTrust > ::ChainActive().Tip()->nChainTrust + (GetBlockProof(*::ChainActive().Tip()) * 6)) {
  LogPrintf("%s: Warning: Found invalid chain at least ~6 blocks longer than our best chain.\nChain state database corruption likely.\n", __func__);
  SetfLargeWorkInvalidChainFound(true);
  - }
  - else
  - {
  + } else {
  SetfLargeWorkInvalidChainFound(false);
  }
  }
  @@ -2537,7 +2535,7 @@ bool CChainState::ActivateBestChainStep(BlockValidationState& state, const CChai
  fBlocksDisconnected = true;
  }
  - // Build list of new blocks to connect.
  + // Build list of new blocks to connect (in descending height order).
  std::vector<CBlockIndex*> vpindexToConnect;
  bool fContinue = true;
  int nHeight = pindexFork ? pindexFork->nHeight : -1;
  