Skip to content

Commit 6416274

Browse files
committed
Fix panic lint
Fix panic lint
1 parent 0a385a4 commit 6416274

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

udp_mux_multi.go

+4-3
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,7 @@ import (
1515
tudp "github.com/pion/transport/v2/udp"
1616
)
1717

18-
var (
19-
errPortBalanceRequireConnCount = errors.New("Port balance requires UDPMux implements MuxConnCount interface")
20-
)
18+
var errPortBalanceRequireConnCount = errors.New("Port balance requires UDPMux implements MuxConnCount interface")
2119

2220
// MultiUDPMuxDefault implements both UDPMux and AllConnsGetter,
2321
// allowing users to pass multiple UDPMux instances to the ICE agent
@@ -74,7 +72,10 @@ type multipleUDPMuxDefaultParams struct {
7472
// uses the provided UDPMux instances.
7573
func NewMultiUDPMuxDefault(muxes ...UDPMux) *MultiUDPMuxDefault {
7674
mux, err := NewMultiUDPMuxDefaultWithOptions(muxes)
75+
// The error should always be nil as no options given to NewMultiUDPMuxDefaultWithOptions that it
76+
// only return error when port balance enabled but mux don't support MuxConnCount.
7777
if err != nil {
78+
//nolint:forbidigo
7879
panic(err)
7980
}
8081
return mux

0 commit comments

Comments
 (0)