<div dir="ltr"><div class="gmail_quote">Hi Francois,</div><div class="gmail_quote"><span><br></span></div><div class="gmail_quote">On Fri, May 19, 2017 at 4:40 PM, Francois Cartegnie <span dir="ltr"><<a href="mailto:fcvlcdev@free.fr" target="_blank">fcvlcdev@free.fr</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid"><div class="gmail-HOEnZb"><div class="gmail-h5">Le 19/05/2017 à 10:31, Zhao Zhili a écrit :<br>
<br>
> +            {<br>
> +#define H264_MAX_DPB_SIZE   18<br>
> +                dpb_size = H264_MAX_DPB_SIZE;<br>
> +<br>
> +                int profile = -1;<br>
> +                int level = -1;<br>
> +<br>
> +                profile = p_dec->fmt_in.i_profile;<br>
> +                level = p_dec->fmt_in.i_level;<br>
> +                if( profile == -1 || level == -1 )<br>
> +                {<br>
> +                    if( h264_get_profile_level( &p_dec->fmt_in, &profile,<br>
> &level, NULL ) == false )<br>
> +                    {<br>
> +                        msg_Warn( p_dec, "get profile and level failed" );<br>
> +                        break;<br>
> +                    }<br>
> +                }<br>
> +                msg_Dbg( p_dec, "H264 profile %d, level %d", profile,<br>
> level );<br>
> +<br>
> +#define PROFILE_H264_MAIN                 77<br>
> +#define PROFILE_H264_HIGH                 100<br>
> +                if( profile != PROFILE_H264_MAIN && profile !=<br>
> PROFILE_H264_HIGH )<br>
> +                    break;<br>
<br>
</div></div>Unfortunately, because of possible configuration changes, that max DPB<br>
will always be set to max.<br>
<br>
And in all cases, dpb value computation does not belong to core, and<br>
cannot be correct if relying on extradata.<br>
<br></blockquote><div><br></div><div>Thanks for your review. I know this patch is ugly and not work in many cases.</div><div>I just wonder how to reduce the memory usage when playing ultra high definition</div><div>videos. Since ultra high definition videos <span><span>have smaller dpb_size, a lot of memory</span></span></div><div><span><span>can be saved. It would be better if we can use the number of reference frames</span></span></div><div><span><span>directly. I don't know how to make this work and don't break anything. </span></span><span><span>Any</span></span></div><div><span><span>suggestions would be appreciated.<span></span><br></span></span></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid">
Francois<br>
______________________________<wbr>_________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" target="_blank" rel="noreferrer">https://mailman.videolan.org/<wbr>listinfo/vlc-devel</a><br></blockquote></div></div>