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

Merge pull request #2995 from lucas-clemente/simplify-tp-parsing-switch

Browse Source 
simplify switch statement in the transport parameter parser
This commit is contained in:
Marten Seemann
2021-01-15 17:05:59 +08:00
committed by GitHub
parent b50933737d 8e94f08ee5
commit cafa7c45d0
1 changed files with 7 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
internal/wire/transport_parameters.go
View File

@@ -123,24 +123,18 @@ func (p *TransportParameters) unmarshal(r *bytes.Reader, sentBy protocol.Perspec
} }
parameterIDs = append(parameterIDs, paramID) parameterIDs = append(parameterIDs, paramID)
switch paramID { switch paramID {
case ackDelayExponentParameterID: case maxIdleTimeoutParameterID,
if err := p.readNumericTransportParameter(r, paramID, int(paramLen)); err != nil { maxUDPPayloadSizeParameterID,
return err initialMaxDataParameterID,
} initialMaxStreamDataBidiLocalParameterID,
case maxAckDelayParameterID:
if err := p.readNumericTransportParameter(r, paramID, int(paramLen)); err != nil {
return err
}
case initialMaxStreamDataBidiLocalParameterID,
initialMaxStreamDataBidiRemoteParameterID, initialMaxStreamDataBidiRemoteParameterID,
initialMaxStreamDataUniParameterID, initialMaxStreamDataUniParameterID,
initialMaxDataParameterID,
initialMaxStreamsBidiParameterID, initialMaxStreamsBidiParameterID,
initialMaxStreamsUniParameterID, initialMaxStreamsUniParameterID,
maxIdleTimeoutParameterID, maxAckDelayParameterID,
maxUDPPayloadSizeParameterID,
activeConnectionIDLimitParameterID, activeConnectionIDLimitParameterID,
maxDatagramFrameSizeParameterID: maxDatagramFrameSizeParameterID,
ackDelayExponentParameterID:
if err := p.readNumericTransportParameter(r, paramID, int(paramLen)); err != nil { if err := p.readNumericTransportParameter(r, paramID, int(paramLen)); err != nil {
return err return err
} }