refactor reduce useless var

q191201771 · Jun 21, 2024 · f31e75b · f31e75b
1 parent f5f9f0b
commit f31e75b
Show file tree

Hide file tree

Showing 2 changed files with 129 additions and 130 deletions.
diff --git a/pkg/mpegts/mpegts.go b/pkg/mpegts/mpegts.go
@@ -9,130 +9,7 @@
 package mpegts
 
 // MPEG: Moving Picture Experts Group
-
-// FixedFragmentHeader 每个TS文件都以固定的PAT，PMT开始
-// TODO(chef): [refactor] 考虑去掉这个fixed变量，目前只在测试中使用 [202303]
-var FixedFragmentHeader = []byte{
-	/* TS */
-	0x47, 0x40, 0x00, 0x10, 0x00,
-	/* PSI */
-	0x00, 0xb0, 0x0d, 0x00, 0x01, 0xc1, 0x00, 0x00,
-	/* PAT */
-	0x00, 0x01, 0xf0, 0x01,
-	/* CRC */
-	0x2e, 0x70, 0x19, 0x05,
-
-	/* stuffing 167 bytes */
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-
-	/* TS */
-	0x47, 0x50, 0x01, 0x10, 0x00,
-	/* PSI */
-	0x02, 0xb0, 0x17, 0x00, 0x01, 0xc1, 0x00, 0x00,
-	/* PMT */
-	0xe1, 0x00,
-	0xf0, 0x00,
-	0x1b, 0xe1, 0x00, 0xf0, 0x00, /* avc epid 256 */
-	0x0f, 0xe1, 0x01, 0xf0, 0x00, /* aac epid 257 */
-	/* CRC */
-	0x2f, 0x44, 0xb9, 0x9b, /* crc for aac */
-	/* stuffing 157 bytes */
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-}
-
-// FixedFragmentHeaderHevc 每个TS文件都以固定的PAT，PMT开始
-// TODO(chef): [refactor] 考虑去掉这个fixed变量，目前只在测试中使用 [202303]
-var FixedFragmentHeaderHevc = []byte{
-	/* TS */
-	0x47, 0x40, 0x00, 0x10, 0x00,
-	/* PSI */
-	0x00, 0xb0, 0x0d, 0x00, 0x01, 0xc1, 0x00, 0x00,
-	/* PAT */
-	0x00, 0x01, 0xf0, 0x01,
-	/* CRC */
-	0x2e, 0x70, 0x19, 0x05,
-
-	/* stuffing 167 bytes */
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-
-	/* TS */
-	0x47, 0x50, 0x01, 0x10, 0x00,
-	/* PSI */
-	0x02, 0xb0, 0x17, 0x00, 0x01, 0xc1, 0x00, 0x00,
-	/* PMT */
-	0xe1, 0x00,
-	0xf0, 0x00,
-	//0x1b, 0xe1, 0x00, 0xf0, 0x00, /* avc epid 256 */
-	0x24, 0xe1, 0x00, 0xf0, 0x00,
-	0x0f, 0xe1, 0x01, 0xf0, 0x00, /* aac epid 257 */
-	/* CRC */
-	//0x2f, 0x44, 0xb9, 0x9b, /* crc for aac */
-	0xc7, 0x72, 0xb7, 0xcb,
-	/* stuffing 157 bytes */
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
-}
+// <iso13818-1.pdf>
 
 // TS Packet Header
 const (
@@ -154,7 +31,7 @@ const (
 
 // PMT
 const (
-	// -----------------------------------------------------------------------------
+	// StreamTypeUnknown ... -----------------------------------------------------------------------------
 	// <iso13818-1.pdf> <Table 2-29 Stream type assignments> <page 66/174>
 	// 0x0F AAC  (ISO/IEC 13818-7 Audio with ADTS transport syntax)
 	// 0x1B AVC  (video stream as defined in ITU-T Rec. H.264 | ISO/IEC 14496-10 Video)
@@ -169,7 +46,7 @@ const (
 
 // PES
 const (
-	// StreamIdAudio -----------------------------------------------------------------
+	// StreamIdAudio ... -----------------------------------------------------------------
 	// <iso13818-1.pdf> <Table 2-18-Stream_id assignments> <page 52/174>
 	// -----------------------------------------------------------------
 	StreamIdAudio uint8 = 192 // 110x xxxx 0xC0

diff --git a/pkg/mpegts/mpegts_test.go b/pkg/mpegts/mpegts_test.go
@@ -16,18 +16,140 @@ import (
 	"github.com/q191201771/lal/pkg/mpegts"
 )
 
+// FixedFragmentHeader 每个TS文件都以固定的PAT，PMT开始
+var FixedFragmentHeader = []byte{
+	/* TS */
+	0x47, 0x40, 0x00, 0x10, 0x00,
+	/* PSI */
+	0x00, 0xb0, 0x0d, 0x00, 0x01, 0xc1, 0x00, 0x00,
+	/* PAT */
+	0x00, 0x01, 0xf0, 0x01,
+	/* CRC */
+	0x2e, 0x70, 0x19, 0x05,
+
+	/* stuffing 167 bytes */
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+
+	/* TS */
+	0x47, 0x50, 0x01, 0x10, 0x00,
+	/* PSI */
+	0x02, 0xb0, 0x17, 0x00, 0x01, 0xc1, 0x00, 0x00,
+	/* PMT */
+	0xe1, 0x00,
+	0xf0, 0x00,
+	0x1b, 0xe1, 0x00, 0xf0, 0x00, /* avc epid 256 */
+	0x0f, 0xe1, 0x01, 0xf0, 0x00, /* aac epid 257 */
+	/* CRC */
+	0x2f, 0x44, 0xb9, 0x9b, /* crc for aac */
+	/* stuffing 157 bytes */
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+}
+
+// FixedFragmentHeaderHevc 每个TS文件都以固定的PAT，PMT开始
+var FixedFragmentHeaderHevc = []byte{
+	/* TS */
+	0x47, 0x40, 0x00, 0x10, 0x00,
+	/* PSI */
+	0x00, 0xb0, 0x0d, 0x00, 0x01, 0xc1, 0x00, 0x00,
+	/* PAT */
+	0x00, 0x01, 0xf0, 0x01,
+	/* CRC */
+	0x2e, 0x70, 0x19, 0x05,
+
+	/* stuffing 167 bytes */
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+
+	/* TS */
+	0x47, 0x50, 0x01, 0x10, 0x00,
+	/* PSI */
+	0x02, 0xb0, 0x17, 0x00, 0x01, 0xc1, 0x00, 0x00,
+	/* PMT */
+	0xe1, 0x00,
+	0xf0, 0x00,
+	//0x1b, 0xe1, 0x00, 0xf0, 0x00, /* avc epid 256 */
+	0x24, 0xe1, 0x00, 0xf0, 0x00,
+	0x0f, 0xe1, 0x01, 0xf0, 0x00, /* aac epid 257 */
+	/* CRC */
+	//0x2f, 0x44, 0xb9, 0x9b, /* crc for aac */
+	0xc7, 0x72, 0xb7, 0xcb,
+	/* stuffing 157 bytes */
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+	0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+}
+
 func TestMpegts(t *testing.T) {
 	innertest.Entry(t)
 }
 
 func TestParseFixedTsPacket(t *testing.T) {
-	h := mpegts.ParseTsPacketHeader(mpegts.FixedFragmentHeader)
+	h := mpegts.ParseTsPacketHeader(FixedFragmentHeader)
 	mpegts.Log.Debugf("%+v", h)
-	pat := mpegts.ParsePat(mpegts.FixedFragmentHeader[5:])
+	pat := mpegts.ParsePat(FixedFragmentHeader[5:])
 	mpegts.Log.Debugf("%+v", pat)
 
-	h = mpegts.ParseTsPacketHeader(mpegts.FixedFragmentHeaderHevc[188:])
+	h = mpegts.ParseTsPacketHeader(FixedFragmentHeaderHevc[188:])
 	mpegts.Log.Debugf("%+v", h)
-	pmt := mpegts.ParsePmt(mpegts.FixedFragmentHeader[188+5:])
+	pmt := mpegts.ParsePmt(FixedFragmentHeader[188+5:])
 	mpegts.Log.Debugf("%+v", pmt)
 }