<div dir="ltr">Daniel,<br><div class="gmail_extra"><br><div class="gmail_quote">On Sun, May 31, 2015 at 10:03 PM, Daniel Kamil Kozar <span dir="ltr"><<a href="mailto:dkk089@gmail.com" target="_blank">dkk089@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">b_mpeg2 in dvbpsi_vstream_dr_t corresponds to what the H.222.0 refers to<br>
as the "MPEG_1_only_flag". If the flag is not set, it means that no<br>
extra data is contained at the end of the descriptor. The code in<br>
dr_02.c did exactly the opposite.<br>
---<br>
src/descriptors/dr_02.c | 2 +-<br>
1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/src/descriptors/dr_02.c b/src/descriptors/dr_02.c<br>
index 774a1c0..1c35d83 100644<br>
--- a/src/descriptors/dr_02.c<br>
+++ b/src/descriptors/dr_02.c<br>
@@ -60,7 +60,7 @@ dvbpsi_vstream_dr_t * dvbpsi_DecodeVStreamDr(dvbpsi_descriptor_t * p_descriptor)<br>
if(!p_decoded) return NULL;<br>
<br>
/* Decode data and check the length */<br>
- p_decoded->b_mpeg2 = !((p_descriptor->p_data[0] & 0x04) ? true : false);<br>
+ p_decoded->b_mpeg2 = (p_descriptor->p_data[0] & 0x04) ? true : false;<br>
if( (!p_decoded->b_mpeg2 && (p_descriptor->i_length != 1))<br>
|| (p_decoded->b_mpeg2 && (p_descriptor->i_length != 3)))<br>
{<br>
<span class=""><font color="#888888">--<br>
2.4.2<br>
<br></font></span></blockquote><div><br>Could you double check the specs, because it would mean a revert of 25cdb9fd, which I believe is correct in the mentioned specs.<br>That H.222.0 and " ISO/IEC 13818-1 : 2000, table 2-40 and 2.6.3" contradict each other is quite unlikely IMHO.<br><br>$> git show 25cdb9fd<br>commit 25cdb9fd7ce804ec69b91fa1b0887435c474b32d<br>Author: Jean-Baptiste Kempf <<a href="mailto:jb@videolan.org">jb@videolan.org</a>><br>Date: Tue Aug 12 04:04:47 2014 +0200<br><br> Correctly flag mpeg2 video data in video_stream_descriptor<br> <br> See ISO/IEC 13818-1 : 2000, table 2-40 and 2.6.3<br> <br> See <a href="https://trac.videolan.org/vlc/ticket/4200">https://trac.videolan.org/vlc/ticket/4200</a><br> <br> Signed-off-by: Jean-Paul Saman <<a href="mailto:jpsaman@videolan.org">jpsaman@videolan.org</a>><br><br>diff --git a/src/descriptors/dr_02.c b/src/descriptors/dr_02.c<br>index 1c35d83..774a1c0 100644<br>--- a/src/descriptors/dr_02.c<br>+++ b/src/descriptors/dr_02.c<br>@@ -60,7 +60,7 @@ dvbpsi_vstream_dr_t * dvbpsi_DecodeVStreamDr(dvbpsi_descriptor_t * p_descriptor)<br> if(!p_decoded) return NULL;<br> <br> /* Decode data and check the length */<br>- p_decoded->b_mpeg2 = (p_descriptor->p_data[0] & 0x04) ? true : false;<br>+ p_decoded->b_mpeg2 = !((p_descriptor->p_data[0] & 0x04) ? true : false);<br> if( (!p_decoded->b_mpeg2 && (p_descriptor->i_length != 1))<br> || (p_decoded->b_mpeg2 && (p_descriptor->i_length != 3)))<br> {<br> <br></div><div>Kind regards,<br><br></div><div>Jean-Paul Saman<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class=""><font color="#888888">
_______________________________________________<br>
libdvbpsi-devel mailing list<br>
<a href="mailto:libdvbpsi-devel@videolan.org">libdvbpsi-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/libdvbpsi-devel" target="_blank">https://mailman.videolan.org/listinfo/libdvbpsi-devel</a><br>
</font></span></blockquote></div><br></div></div>