On Sat, Sep 17, 2016 at 01:09:02PM +0800, Steven Liu wrote:
[...]
> @@ -867,6 +886,16 @@ static int hls_write_header(AVFormatContext *s)
>      for (i = 0; i < s->nb_streams; i++) {
>          AVStream *inner_st;
>          AVStream *outer_st = s->streams[i];
> +
> +        if (hls->max_seg_size > 0) {
> +            if ((outer_st->codecpar->codec_type == AVMEDIA_TYPE_VIDEO) &&
> +                outer_st->codecpar->bit_rate > hls->max_seg_size) {
> +                av_log(s, AV_LOG_WARNING, "Your video bitrate is bigger than 
> hls_segment_size, "
> +                       "%lld > %lld ( video birate > hls_segment_size ),the 
> result maybe not you want.",
> +                       outer_st->codecpar->bit_rate, hls->max_seg_size);

%lld is the wrong type for *int64_t

The compiler should produce a warning for this, though probably not
every copiler does
please make sure your changes add no warnings

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The bravest are surely those who have the clearest vision
of what is before them, glory and danger alike, and yet
notwithstanding go out to meet it. -- Thucydides

Attachment: signature.asc
Description: Digital signature

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

Reply via email to