Skip to content

Commit

Permalink
simplify updateWritableStreams
Browse files Browse the repository at this point in the history
Summary: These are all duplicated from `removeWritable`.

Reviewed By: meleshuk, mjoras

Differential Revision: D68017903

fbshipit-source-id: 5f780a0b7d7f2d2d5d4c6430510501dd11eae2a2
  • Loading branch information
pfarcasanu authored and facebook-github-bot committed Jan 17, 2025
1 parent 5dff5c5 commit c64c82a
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions quic/state/QuicStreamManager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -667,15 +667,6 @@ void QuicStreamManager::updateWritableStreams(QuicStreamState& stream) {
CHECK(stream.lossBuffer.empty());
CHECK(stream.lossBufMetas.empty());
removeWritable(stream);
writableStreams_.erase(stream.id);
writableDSRStreams_.erase(stream.id);
lossStreams_.erase(stream.id);
lossDSRStreams_.erase(stream.id);
if (stream.isControl) {
controlWriteQueue_.erase(stream.id);
} else {
writeQueue_.erase(stream.id);
}
return;
}
if (stream.hasWritableData()) {
Expand Down

0 comments on commit c64c82a

Please sign in to comment.