Remove redundant clause in if statement for resets

Summary: `stream.finalReadOffset` is a `folly::Optional` that we're setting with an actual value right before the statement in which I'm making the modification.

Reviewed By: hanidamlaj

Differential Revision: D65213768

fbshipit-source-id: 4c95ec8dc03fef69e974d4131ce6f98262151c02

quic/state/stream/StreamStateFunctions.cpp

@@ -50,8 +50,7 @@ void onResetQuicStream(QuicStreamState& stream, const RstStreamFrame& frame) {
   stream.readBuffer.clear();
   stream.finalReadOffset = frame.offset;
   stream.streamReadError = frame.errorCode;
-  bool appReadAllBytes = stream.finalReadOffset &&
-      stream.currentReadOffset > *stream.finalReadOffset;
+  bool appReadAllBytes = stream.currentReadOffset > *stream.finalReadOffset;
   if (!appReadAllBytes) {
     // If the currentReadOffset > finalReadOffset we have already processed
     // all the bytes until FIN, so we don't need to do anything for the read