Ping for the patch set.

> On Oct 21, 2021, at 4:15 PM, Zhao Zhili <quinkbl...@foxmail.com> wrote:
> 
> There is a little chance that user specified contradicted options
> like -streaming 0 -ldash 1, however, it's more likely that user
> didn't know or forgot to enable streaming for ldash. So enabling
> streaming automatically to make the feature easier to use, similar
> like enable FF_MOV_FLAG_FRAGMENT/EMPTY_MOOV/DEFAULT_BASE_MOOF and
> so on for FF_MOV_FLAG_CMAF.
> ---
> libavformat/dashenc.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c
> index 06bbf36c95..0370a482fb 100644
> --- a/libavformat/dashenc.c
> +++ b/libavformat/dashenc.c
> @@ -1406,8 +1406,8 @@ static int dash_init(AVFormatContext *s)
>     }
> 
>     if (c->ldash && !c->streaming) {
> -        av_log(s, AV_LOG_WARNING, "LDash option will be ignored as streaming 
> is not enabled\n");
> -        c->ldash = 0;
> +        av_log(s, AV_LOG_WARNING, "Enabling streaming as LDash is 
> enabled\n");
> +        c->streaming = 1;
>     }
> 
>     if (c->target_latency && !c->streaming) {
> -- 
> 2.31.1
> 

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to