osmith
/
wireshark
forked from osmocom/wireshark
1
0
Fork 0
Code Pull Requests Releases Wiki Activity

VP8: some cosmetics from RFC 7741

This commit is contained in:
Valerii Zapodovnikov 2021-03-12 17:05:22 +00:00 committed by Wireshark GitLab Utility
parent dabf0af6b7
commit cc6a45420a
1 changed files with 13 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
epan/dissectors/packet-vp8.c
View File

@ -11,7 +11,7 @@
/*
* RFC 6386 - VP8 Data Format and Decoding Guide
* RFC 7741 - RTP Payload Format for NP8 Video
* RFC 7741 - RTP Payload Format for VP8 Video
*/
@ -133,12 +133,12 @@ static const true_false_string vp8_x_bit_vals = {
};
static const true_false_string vp8_r_bit_vals = {
"Reserved for future use (Error should be zero)",
"Reserved for future use (error: should be zero)",
"Reserved for future use"
};
static const true_false_string vp8_n_bit_vals = {
"Non referenced frame",
"Non-reference frame",
"Reference frame"
};
@ -158,13 +158,13 @@ static const true_false_string vp8_l_bit_vals = {
};
static const true_false_string vp8_t_bit_vals = {
"TID (temporal layer index) byte present",
"TID (temporal layer index) byte not present"
"TID (temporal layer index) present",
"TID (temporal layer index) not present"
};
static const true_false_string vp8_k_bit_vals = {
"TID/KEYIDX byte present",
"TID/KEYIDX byte not present"
"KEYIDX present",
"KEYIDX not present"
};
static const true_false_string vp8_hdr_frametype_vals = {
@ -215,11 +215,11 @@ The first octets after the RTP header are the VP8 payload descriptor,
0 1 2 3 4 5 6 7
+-+-+-+-+-+-+-+-+
|X|R|N|S|PartID | (REQUIRED)
|X|R|N|S|R| PID | (REQUIRED), second R bit is parsed as part of PID
+-+-+-+-+-+-+-+-+
X: |I|L|T|K| RSV | (OPTIONAL)
+-+-+-+-+-+-+-+-+
I: | PictureID | (OPTIONAL)
I: |M| PictureID | (OPTIONAL)
+-+-+-+-+-+-+-+-+
L: | TL0PICIDX | (OPTIONAL)
+-+-+-+-+-+-+-+-+
@ -447,7 +447,7 @@ proto_register_vp8(void)
NULL, HFILL }
},
{ &hf_vp8_pld_part_id,
{ "Part Id", "vp8.pld.partid",
{ "2nd R bit and Part Id", "vp8.pld.partid",
FT_UINT8, BASE_DEC, NULL, BIT_5678_MASK,
NULL, HFILL }
},
@ -477,12 +477,12 @@ proto_register_vp8(void)
NULL, HFILL }
},
{ &hf_vp8_pld_picture_id,
{ "Picture Id", "vp8.pld.pictureid",
{ "Picture ID", "vp8.pld.pictureid",
FT_UINT8, BASE_DEC, NULL, BIT_NO_MASK,
NULL, HFILL }
},
{ &hf_vp8_pld_extended_picture_id,
{ "Extended Picture Id", "vp8.pld.pictureid",
{ "Extended Picture ID", "vp8.pld.pictureid",
FT_UINT8, BASE_DEC, NULL, BIT_EXT_PICTURE_MASK,
NULL, HFILL }
},
@ -497,7 +497,7 @@ proto_register_vp8(void)
NULL, HFILL }
},
{ &hf_vp8_pld_y_bit,
{ "1 Lay Sync Bit (Y)", "vp8.pld.y",
{ "1 layer sync bit (Y)", "vp8.pld.y",
FT_BOOLEAN, 8, NULL, BIT_3_MASK,
NULL, HFILL }
},