<div dir="ltr">Illka, <br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 25, 2015 at 8:15 AM, Ilkka Ollakka <span dir="ltr"><<a href="mailto:ileoo@videolan.org" target="_blank">ileoo@videolan.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, Sep 21, 2015 at 06:24:12PM +0300, Ilkka Ollakka wrote:<br>
> On Mon, Sep 21, 2015 at 10:54:01AM +0200, Francois Cartegnie wrote:<br>
> > Hi Ilkka,<br>
<br>
</span>Hi again,<br>
<span class=""><br>
<br>
> > Can you confirm your own understanding of 2.4.3.5, especially towards<br>
> > the end, because "after a continuity counter discontinuity" is not clear.<br>
<br>
> If you refer to TS spec, no, I don't think I have that in hand<br>
> currently or if I tought of that any more when cooked the patch.<br>
<br>
</span>Thanks for the specs that people pointed. After briefly reading that<br>
section I think I have following understanding. Not sure if it's<br>
actually the issue you are asking about, and following is quite a lot<br>
just flow of thoughts when trying to create mental image of the thing:<br>
<br>
- If we have discontinuity indicator on in non-PCR pid, we can just not<br>
  care about continuity counter. So we can't know packet drops or others<br>
  using that counter when discontinuity is signaled.<br>
- For PCR pid continuity counter should always be running correctly.<br>
- Every non-PCR pid should have at max 2 packets in row with<br>
  discontinuity_indicator set on<br>
<br>
- After continuity counter discontinuity next packet should be start of<br>
  new video sequence/avc access unit/audio frame. This doesn't actually<br>
  clearly mention that packet with discontinuity signaled should be such<br>
  packet. Assuming if random_access_indicator is set then the same<br>
  packet also contains new frame/block/video/* start.<br>
<br>
  When reading that random_access_indicator description, I interpret it<br>
  in a way that discontinuity signaled packet contains the start of new<br>
  frame.<br>
<br>
  But it's little puzzling on page 23 second last chapter to talk<br>
  about after continuity counter discontinuity there should be start of<br>
  new frame in packet.<br>
<br>
  So based on previous assumptions I made in this mail, I assume that it<br>
  means when pcr pid gets discontinuity signaled it means that same<br>
  packet that signals it, contains start of new frame with new<br>
  timestamps and we reset discontinuity counter checking.<br>
<span class=""><br></span></blockquote><div><br>I read the specs basically the same as your understanding above.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
<br>
> > Seems also the ts demux discontinuity flag on PCR must be handled<br>
> > differently :/<br>
<br>
</span>Looks like I need lots of paper and pens with different colors to try to<br>
get my head around that.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Ilkka Ollakka<br>
Catharsis is something I associate with pornography and crossword puzzles.<br>
                -- Howard Chaykin<br>
</font></span><a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank"></a><br>
<br></blockquote></div>Kind regards,<br></div><div class="gmail_extra">Jean-Paul Saman<br></div></div>