commit be88527b8dca5b1de1dd2fcd891bc12e22bba992
author bnc <bnc@google.com> Thu Jan 16 11:10:48 2020 -0800
committer Copybara-Service <copybara-worker@google.com> Thu Jan 16 11:11:24 2020 -0800
tree 1c5cf3aa3a688fd15585e3e7753e4797d3816825
parent e5e25bc2ec5cab450bd4161cfa17381cf8f74170

Break up conjunctive DCHECK expressions.

DCHECK(A && B); => DCHECK(A); DCHECK(B);

The motivation is more useful output in case of failure.

Note that just like before this change, B is evaluated iff A is true.  Before
this change because of the lazy evaluation rule, afterwards because DCHECK(A)
blows up if A is false.

gfe-relnote: n/a, no functional change.
PiperOrigin-RevId: 290103915
Change-Id: I18bdbfbd5cdf3411b45ec4cbfa7ffebc009ed9a9

quic/core/quic_framer.cc

diff --git a/quic/core/quic_framer.cc b/quic/core/quic_framer.cc
index 8c2f86a..bb048e1 100644
--- a/quic/core/quic_framer.cc
+++ b/quic/core/quic_framer.cc
@@ -5257,8 +5257,8 @@
                                         const QuicStopWaitingFrame& frame,
                                         QuicDataWriter* writer) {
   DCHECK(!VersionHasIetfInvariantHeader(version_.transport_version));
-  DCHECK(frame.least_unacked.IsInitialized() &&
-         header.packet_number >= frame.least_unacked);
+  DCHECK(frame.least_unacked.IsInitialized());
+  DCHECK_GE(header.packet_number, frame.least_unacked);
   const uint64_t least_unacked_delta =
       header.packet_number - frame.least_unacked;
   const uint64_t length_shift = header.packet_number_length * 8;
@@ -6093,8 +6093,8 @@
 void QuicFramer::InferPacketHeaderTypeFromVersion() {
   // This function should only be called when server connection negotiates the
   // version.
-  DCHECK(perspective_ == Perspective::IS_SERVER &&
-         !infer_packet_header_type_from_version_);
+  DCHECK_EQ(perspective_, Perspective::IS_SERVER);
+  DCHECK(!infer_packet_header_type_from_version_);
   infer_packet_header_type_from_version_ = true;
 }