simplify switch statement in the transport parameter parser

quic-go · Jan 11, 2021 · 8e94f08 · 8e94f08
1 parent d1c5297
commit 8e94f08
Showing 1 changed file with 7 additions and 13 deletions.
diff --git a/internal/wire/transport_parameters.go b/internal/wire/transport_parameters.go
@@ -123,24 +123,18 @@ func (p *TransportParameters) unmarshal(r *bytes.Reader, sentBy protocol.Perspec
 		}
 		parameterIDs = append(parameterIDs, paramID)
 		switch paramID {
-		case ackDelayExponentParameterID:
-			if err := p.readNumericTransportParameter(r, paramID, int(paramLen)); err != nil {
-				return err
-			}
-		case maxAckDelayParameterID:
-			if err := p.readNumericTransportParameter(r, paramID, int(paramLen)); err != nil {
-				return err
-			}
-		case initialMaxStreamDataBidiLocalParameterID,
+		case maxIdleTimeoutParameterID,
+			maxUDPPayloadSizeParameterID,
+			initialMaxDataParameterID,
+			initialMaxStreamDataBidiLocalParameterID,
 			initialMaxStreamDataBidiRemoteParameterID,
 			initialMaxStreamDataUniParameterID,
-			initialMaxDataParameterID,
 			initialMaxStreamsBidiParameterID,
 			initialMaxStreamsUniParameterID,
-			maxIdleTimeoutParameterID,
-			maxUDPPayloadSizeParameterID,
+			maxAckDelayParameterID,
 			activeConnectionIDLimitParameterID,
-			maxDatagramFrameSizeParameterID:
+			maxDatagramFrameSizeParameterID,
+			ackDelayExponentParameterID:
 			if err := p.readNumericTransportParameter(r, paramID, int(paramLen)); err != nil {
 				return err
 			}