gr1ffon/quic-go
1
0
Fork 0
forked from quic-go/quic-go
Code Pull Requests Activity

trigger window updates before queueing frames for retransmission

Browse Source 
ref #394
This commit is contained in:
Marten Seemann
2017-01-14 15:22:53 +07:00
parent 71227437ee
commit 1e78104f83
1 changed files with 10 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
session.go
View File

@@ -488,6 +488,16 @@ func (s *Session) sendPacket() error {
var controlFrames []frames.Frame var controlFrames []frames.Frame
// get WindowUpdate frames
// this call triggers the flow controller to increase the flow control windows, if necessary
windowUpdateFrames, err := s.getWindowUpdateFrames()
if err != nil {
return err
}
for _, wuf := range windowUpdateFrames {
controlFrames = append(controlFrames, wuf)
}
// check for retransmissions first // check for retransmissions first
for { for {
retransmitPacket := s.sentPacketHandler.DequeuePacketForRetransmission() retransmitPacket := s.sentPacketHandler.DequeuePacketForRetransmission()
@@ -515,13 +525,6 @@ func (s *Session) sendPacket() error {
} }
} }
windowUpdateFrames, err := s.getWindowUpdateFrames()
if err != nil {
return err
}
for _, wuf := range windowUpdateFrames {
controlFrames = append(controlFrames, wuf)
}
ack := s.receivedPacketHandler.GetAckFrame() ack := s.receivedPacketHandler.GetAckFrame()
if ack != nil { if ack != nil {
controlFrames = append(controlFrames, ack) controlFrames = append(controlFrames, ack)