On Mon, Sep 15, 2014 at 06:34:44AM +0000, Raento Mika wrote: > On 15/09/14 01:50, "Michael Niedermayer" <michae...@gmx.at> wrote: > > >On Sun, Sep 14, 2014 at 03:37:00PM +0300, Mika Raento wrote: [...] > >> @@ -289,17 +314,27 @@ static int hls_write_packet(AVFormatContext *s, > >>AVPacket *pkt) > >> > >> if (can_split && av_compare_ts(pkt->pts - hls->start_pts, > >>st->time_base, > >> end_pts, AV_TIME_BASE_Q) >= 0) { > >> - ret = hls_append_segment(hls, hls->duration); > >> + av_write_frame(oc, NULL); /* Flush any buffered data */ > >> + > >> + hls->size = hls->avf->pb->pos - hls->start_pos; > > > >is it intended to access pos directly instead of using avio_tell() ?
> > > > > >> + int64_t start_pos = hls->avf->pb->pos; > > > >this mixes declarations and statements, some compilers dislike that did you miss the 2 comments above ? [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB No snowflake in an avalanche ever feels responsible. -- Voltaire
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel