On 15/09/14 14:26, "Michael Niedermayer" <michae...@gmx.at> wrote:
>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 ?
I did, bugger. Will re-submit.
>
>
>[...]
>--
>Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
>No snowflake in an avalanche ever feels responsible. -- Voltaire
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
