<div dir="ltr"><span style="font-size:small;text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">> I don't see how this patch can't break encoding with broken streams not</span><br style="font-size:small;text-decoration-style:initial;text-decoration-color:initial"><span style="font-size:small;text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">> sending proper PTS or PTS at all.</span><br><div><span style="font-size:small;text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline"><br></span></div><div><span style="font-size:small;text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">This is just an option. Without enabling it explicitly the behavior is the same as previous.</span></div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jul 26, 2018 at 9:41 AM Francois Cartegnie <<a href="mailto:fcvlcdev@free.fr">fcvlcdev@free.fr</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le 26/07/2018 à 09:34, Paweł Wegner a écrit :<br>
> No, it is not related to the decoder output. Decoder has `skip-frames`<br>
> option as well, but it is not useful since it is<br>
> the encoder which drops frames.<br>
> My problem is that those timestamps are different every time I run the<br>
> code. Looks like they are absolute,<br>
> not relative to the beginning of the video. Is this intended?<br>
> <br>
> On Thu, Jul 26, 2018 at 8:58 AM Steve Lhomme <<a href="mailto:robux4@ycbcr.xyz" target="_blank">robux4@ycbcr.xyz</a>> wrote:<br>
> <br>
>> On 24/07/2018 21:59, Paweł Wegner wrote:<br>
>>> Encoder will encode only one frame per timestamp. Since timestamps may be<br>
>>> inaccurate, skipping frames may render encoder's output nondeterministic.<br>
>>> Added an option which instructs to encode all frames which come out of<br>
>> avcodec.<br>
>><br>
>> Is this related to the output of the avcodec decoder ? Because that's<br>
>> how I understand the commit log and doc of the option. But the option is<br>
>> on the encoder, which may receive frames from other decoders.<br>
<br>
They are scheduled on your system, so that's system clock.<br>
<br>
I don't see how this patch can't break encoding with broken streams not<br>
sending proper PTS or PTS at all.<br>
<br>
<br>
-- <br>
Francois Cartegnie<br>
VideoLAN - VLC Developer<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a></blockquote></div>