Remove unused ClockRate parameter

This parameter doesn't appear to be used anywhere.
pion · Dec 9, 2021 · 885dab7 · 885dab7
1 parent 9c94cc0
commit 885dab7
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
diff --git a/AUTHORS.txt b/AUTHORS.txt
@@ -19,6 +19,7 @@ Hugo Arregui <[email protected]>
 John Bradley <[email protected]>
 Juliusz Chroboczek <[email protected]>
 Kazuyuki Honda <[email protected]>
+Kevin Wang <[email protected]>
 Luke Curley <[email protected]>
 lxb <[email protected]>
 Michael MacDonald <[email protected]>

diff --git a/packetizer.go b/packetizer.go
@@ -23,23 +23,21 @@ type packetizer struct {
 	Payloader        Payloader
 	Sequencer        Sequencer
 	Timestamp        uint32
-	ClockRate        uint32
 	extensionNumbers struct { // put extension numbers in here. If they're 0, the extension is disabled (0 is not a legal extension number)
 		AbsSendTime int // http://www.webrtc.org/experiments/rtp-hdrext/abs-send-time
 	}
 	timegen func() time.Time
 }
 
 // NewPacketizer returns a new instance of a Packetizer for a specific payloader
-func NewPacketizer(mtu uint16, pt uint8, ssrc uint32, payloader Payloader, sequencer Sequencer, clockRate uint32) Packetizer {
+func NewPacketizer(mtu uint16, pt uint8, ssrc uint32, payloader Payloader, sequencer Sequencer) Packetizer {
 	return &packetizer{
 		MTU:         mtu,
 		PayloadType: pt,
 		SSRC:        ssrc,
 		Payloader:   payloader,
 		Sequencer:   sequencer,
 		Timestamp:   globalMathRandomGenerator.Uint32(),
-		ClockRate:   clockRate,
 		timegen:     time.Now,
 	}
 }

diff --git a/packetizer_test.go b/packetizer_test.go
@@ -12,7 +12,7 @@ import (
 func TestPacketizer(t *testing.T) {
 	multiplepayload := make([]byte, 128)
 	// use the G722 payloader here, because it's very simple and all 0s is valid G722 data.
-	packetizer := NewPacketizer(100, 98, 0x1234ABCD, &codecs.G722Payloader{}, NewRandomSequencer(), 90000)
+	packetizer := NewPacketizer(100, 98, 0x1234ABCD, &codecs.G722Payloader{}, NewRandomSequencer())
 	packets := packetizer.Packetize(multiplepayload, 2000)
 
 	if len(packets) != 2 {
@@ -26,7 +26,7 @@ func TestPacketizer(t *testing.T) {
 
 func TestPacketizer_AbsSendTime(t *testing.T) {
 	// use the G722 payloader here, because it's very simple and all 0s is valid G722 data.
-	pktizer := NewPacketizer(100, 98, 0x1234ABCD, &codecs.G722Payloader{}, NewFixedSequencer(1234), 90000)
+	pktizer := NewPacketizer(100, 98, 0x1234ABCD, &codecs.G722Payloader{}, NewFixedSequencer(1234))
 	pktizer.(*packetizer).Timestamp = 45678
 	pktizer.(*packetizer).timegen = func() time.Time {
 		return time.Date(1985, time.June, 23, 4, 0, 0, 0, time.FixedZone("UTC-5", -5*60*60))