Remove unused ClockRate parameter.

This parameter doesn't appear to be used anywhere.
pion · Dec 8, 2021 · 48e8ade · 48e8ade
1 parent 9c94cc0
commit 48e8ade
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 <hugo@decentraland.org>
 John Bradley <jrb@turrettech.com>
 Juliusz Chroboczek <jch@irif.fr>
 Kazuyuki Honda <hakobera@gmail.com>
+Kevin Wang <kevmo314@gmail.com>
 Luke Curley <kixelated@gmail.com>
 lxb <xiangbing.li@uama.com.cn>
 Michael MacDonald <github@macdonald.cx>

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))