<div style="line-height:1.7;color:#000000;font-size:14px;font-family:Arial"><div>Hi,</div><div><br></div><div>I read x264 source code, and have one question:</div><div><br></div><div><a href="https://code.videolan.org/videolan/x264/blob/master/encoder/slicetype.c#L1203">https://code.videolan.org/videolan/x264/blob/master/encoder/slicetype.c#L1203</a></div><div><br></div><div>in the function of calculate_durations(), if <span class="n">cur_frame</span><span class="o">-></span><span class="n">i_dpb_output_delay < 0, </span><span class="n">cur_frame</span><span class="o">-></span><span class="n">i_dpb_output_delay is set to 0, and previous frame's </span>i_cpb_duration is adjusted via '<span class="n">prev_frame</span><span class="o">-></span><span class="n">i_cpb_duration</span> <span class="o">+=</span> <span class="n">cur_frame</span><span class="o">-></span><span class="n">i_dpb_output_delay</span>'.</div><div><br></div><div>Before <span class="n">previous frame's </span>i_cpb_duration is adjusted, <span class="n">cur_frame</span><span class="o">-></span><span class="n">i_dpb_output_delay is set to 0. Therefore, I think, there is no need adjusting </span><span class="n">prev_frame</span><span class="o">-></span><span class="n">i_cpb_duration like </span>'<span class="n">prev_frame</span><span class="o">-></span><span class="n">i_cpb_duration</span> <span class="o">+=</span> <span class="n">cur_frame</span><span class="o">-></span><span class="n">i_dpb_output_delay</span>'.</div><div><br></div><div>Is that right?</div><div><br></div><div>Thanks!</div><div><br></div><div>Regards</div><div><br></div><div>Andrew</div></div><br><br><span title="neteasefooter"><p> </p></span>