On 7/6/2017 3:55 AM, Steven Liu wrote:
> +    if (hls->segment_type == SEGMENT_TYPE_FMP4 && byterange_mode) {
> +        av_log(s, AV_LOG_WARNING, "Have not support fmp4 byterange mode yet 
> now, patch "
> +               "welcome or wait to support it, change the hls_segment_type 
> to mpegts now.\n");
> +        hls->segment_type = SEGMENT_TYPE_MPEGTS;
> +    }

It should hard-fail, and definitely not just continue on with MPEG-TS segments. 
That's
really unexpected behavior for a user.

It should return AVERROR_PATCHWELCOME, probably.

Do you have plans to implement the functionality in the future?

Cheers,
- Derek
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to