Fix oggwriter creating invalid first page

previousTimestamp default value is not 0 but 1
pion · Nov 18, 2020 · 1ffa87e · 1ffa87e
1 parent 32c082f
commit 1ffa87e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/README.md b/README.md
@@ -212,6 +212,7 @@ Check out the **[contributing wiki](https://github.com/pion/webrtc/wiki/Contribu
 * [Sean Knight](https://github.com/SeanKnight)
 * [o0olele](https://github.com/o0olele)
 * [Bo Shi](https://github.com/bshimc)
+* [Suzuki Takeo](https://github.com/BambooTuna)
 
 ### License
 MIT License - see [LICENSE](LICENSE) for full text
diff --git a/pkg/media/oggwriter/oggwriter.go b/pkg/media/oggwriter/oggwriter.go
@@ -182,7 +182,7 @@ func (i *OggWriter) WriteRTP(packet *rtp.Packet) error {
 	payload := opusPacket.Payload[0:]
 
 	// Should be equivalent to sampleRate * duration
-	if i.previousTimestamp != 0 {
+	if i.previousTimestamp != 1 {
 		increment := packet.Timestamp - i.previousTimestamp
 		i.previousGranulePosition += uint64(increment)
 	}