[FFmpeg-devel] [PATCH 1/2] add option hls_init_time to set init hls window segment

Steven Liu lingjiujianke at gmail.com
Sat Aug 27 17:42:25 EEST 2016


2016-08-27 19:15 GMT+08:00 Steven Liu <lingjiujianke at gmail.com>:

>
>
> 2016-08-27 18:25 GMT+08:00 Michael Niedermayer <michael at niedermayer.cc>:
>
>> On Sat, Aug 27, 2016 at 07:44:01AM +0800, Steven Liu wrote:
>> > 2016-08-27 4:49 GMT+08:00 Michael Niedermayer <michael at niedermayer.cc>:
>> >
>> > > On Fri, Aug 26, 2016 at 05:30:41PM +0800, Steven Liu wrote:
>> > > > recover segments duration time by hls_time after init hls window.
>> > > > This is reuqested by Ibrahim Tachijian
>> > > >
>> > > > Signed-off-by: LiuQi <liuqi at gosun.com>
>> > > > ---
>> > > >  libavformat/hlsenc.c | 14 +++++++++++++-
>> > > >  1 file changed, 13 insertions(+), 1 deletion(-)
>> > > [...]
>> > > >  hlsenc.c |   14 +++++++++++++-
>> > > >  1 file changed, 13 insertions(+), 1 deletion(-)
>> > > > f38066079d46206d1bc970f026a7a9de25ace8d1
>> 0001-add-option-hls_init_time-
>> > > to-set-init-hls-window-segm.patch
>> > > > From 937e5301414f1c43c4c564ee59a4dfeee690293c Mon Sep 17 00:00:00
>> 2001
>> > > > From: Steven Liu <lingjiujianke at gmail.com>
>> > > > Date: Fri, 26 Aug 2016 14:34:58 +0800
>> > > > Subject: [PATCH 1/2] add option hls_init_time to set init hls window
>> > > segment
>> > > >  duration
>> > > >
>> > > > And recover segments duration time by hls_time after init hls
>> window.
>> > > > This is reuqested by Ibrahim Tachijian
>> > > >
>> > > > Signed-off-by: LiuQi <liuqi at gosun.com>
>> > > > ---
>> > > >  libavformat/hlsenc.c | 14 +++++++++++++-
>> > > >  1 file changed, 13 insertions(+), 1 deletion(-)
>> > > >
>> > > > diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
>> > > > index e65f002..f5ceb60 100644
>> > > > --- a/libavformat/hlsenc.c
>> > > > +++ b/libavformat/hlsenc.c
>> > > > @@ -85,6 +85,7 @@ typedef struct HLSContext {
>> > > >      AVFormatContext *vtt_avf;
>> > > >
>> > > >      float time;            // Set by a private option.
>> > > > +    float init_time;            // Set by a private option.
>> > > >      int max_nb_segments;   // Set by a private option.
>> > > >      int  wrap;             // Set by a private option.
>> > >
>> > > the comment looks oddly aligned, it should be at the same horizontal
>> > > position as teh others
>> > >
>> > >
>> > > >      uint32_t flags;        // enum HLSFlags
>> > > > @@ -706,7 +707,7 @@ static int hls_write_header(AVFormatContext *s)
>> > > >      int vtt_basename_size;
>> > > >
>> > > >      hls->sequence       = hls->start_sequence;
>> > > > -    hls->recording_time = hls->time * AV_TIME_BASE;
>> > > > +    hls->recording_time = (hls->init_time ? hls->init_time :
>> hls->time)
>> > > * AV_TIME_BASE;
>> > > >      hls->start_pts      = AV_NOPTS_VALUE;
>> > > >
>> > > >      if (hls->format_options_str) {
>> > > > @@ -860,9 +861,19 @@ static int hls_write_packet(AVFormatContext
>> *s,
>> > > AVPacket *pkt)
>> > > >      AVStream *st = s->streams[pkt->stream_index];
>> > > >      int64_t end_pts = hls->recording_time * hls->number;
>> > > >      int is_ref_pkt = 1;
>> > >
>> > > > +    int init_list_dur = 0;
>> > > > +    int after_init_list_dur = 0;
>> > >
>> > > the initialized value is not used
>> > > also the declaration can be moved into the if() below
>> > >
>> > > >      int ret, can_split = 1;
>> > > >      int stream_index = 0;
>> > > >
>> > > > +    if (hls->sequence - hls->nb_entries > hls->start_sequence &&
>> > > hls->init_time > 0) {
>> > > > +        /* reset end_pts, hls->recording_time at end of the init
>> hls
>> > > list */
>> > > > +        init_list_dur = hls->init_time * hls->nb_entries *
>> AV_TIME_BASE;
>> > > > +        after_init_list_dur = (hls->sequence - hls->nb_entries ) *
>> > > hls->time * AV_TIME_BASE;
>> > > > +        hls->recording_time = hls->time * AV_TIME_BASE;
>> > > > +        end_pts = init_list_dur + after_init_list_dur ;
>> > > > +    }
>> > > > +
>> > > >      if( st->codecpar->codec_type == AVMEDIA_TYPE_SUBTITLE ) {
>> > > >          oc = hls->vtt_avf;
>> > > >          stream_index = 0;
>> > > > @@ -972,6 +983,7 @@ static int hls_write_trailer(struct
>> AVFormatContext
>> > > *s)
>> > > [...]
>> > >
>> > >
>> > > --
>> > > Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC7
>> 87040B0FAB
>> > >
>> > > Let us carefully observe those good qualities wherein our enemies
>> excel us
>> > > and endeavor to excel them, by avoiding what is faulty, and imitating
>> what
>> > > is excellent in them. -- Plutarch
>> > >
>> > > _______________________________________________
>> > > ffmpeg-devel mailing list
>> > > ffmpeg-devel at ffmpeg.org
>> > > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>> > >
>> > > Update patch
>>
>> >  hlsenc.c |   14 +++++++++++++-
>> >  1 file changed, 13 insertions(+), 1 deletion(-)
>> > 2feb683487b70430f2ea423d2678d7f30d0c3079
>> 0001-add-option-hls_init_time-to-set-init-hls-window-segm.patch
>> > From dfadb857714867613afebe1a740a8e7fccaa5443 Mon Sep 17 00:00:00 2001
>> > From: Steven Liu <lingjiujianke at gmail.com>
>> > Date: Sat, 27 Aug 2016 07:32:40 +0800
>> > Subject: [PATCH 1/2] add option hls_init_time to set init hls window
>> segment
>> >  duration
>> >
>> > recover segments duration time by hls_time after init hls window.
>> > This is reuqested by Ibrahim Tachijian
>> >
>> > Reviewed-by: Michael Niedermayer <michael at niedermayer.cc>
>> > Signed-off-by: LiuQi <liuqi at gosun.com>
>> > ---
>> >  libavformat/hlsenc.c | 14 +++++++++++++-
>> >  1 file changed, 13 insertions(+), 1 deletion(-)
>> >
>> > diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
>> > index e65f002..95da86c 100644
>> > --- a/libavformat/hlsenc.c
>> > +++ b/libavformat/hlsenc.c
>> > @@ -85,6 +85,7 @@ typedef struct HLSContext {
>> >      AVFormatContext *vtt_avf;
>> >
>> >      float time;            // Set by a private option.
>> > +    float init_time;       // Set by a private option.
>> >      int max_nb_segments;   // Set by a private option.
>> >      int  wrap;             // Set by a private option.
>> >      uint32_t flags;        // enum HLSFlags
>> > @@ -706,7 +707,7 @@ static int hls_write_header(AVFormatContext *s)
>> >      int vtt_basename_size;
>> >
>> >      hls->sequence       = hls->start_sequence;
>> > -    hls->recording_time = hls->time * AV_TIME_BASE;
>> > +    hls->recording_time = (hls->init_time ? hls->init_time :
>> hls->time) * AV_TIME_BASE;
>> >      hls->start_pts      = AV_NOPTS_VALUE;
>> >
>> >      if (hls->format_options_str) {
>>
>> > @@ -863,6 +864,16 @@ static int hls_write_packet(AVFormatContext *s,
>> AVPacket *pkt)
>> >      int ret, can_split = 1;
>> >      int stream_index = 0;
>> >
>> > +    if (hls->sequence - hls->nb_entries > hls->start_sequence &&
>> hls->init_time > 0) {
>> > +        int init_list_dur = 0;
>> > +        int after_init_list_dur = 0;
>> > +        /* reset end_pts, hls->recording_time at end of the init hls
>> list */
>> > +        init_list_dur = hls->init_time * hls->nb_entries *
>> AV_TIME_BASE;
>> > +        after_init_list_dur = (hls->sequence - hls->nb_entries ) *
>> hls->time * AV_TIME_BASE;
>>
>> the first assignment does nothing
>> int a = 0;
>> a = 5;
>> is the same as
>> int a = 5
>>
>> [...]
>>
>> --
>> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>>
>> Breaking DRM is a little like attempting to break through a door even
>> though the window is wide open and the only thing in the house is a bunch
>> of things you dont want and which you would get tomorrow for free anyway
>>
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>> refined the patch and update patch by Michael's suggestion
>
Ignore prev patch, this is update patch: Update commit Author
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-add-option-hls_init_time-to-set-init-hls-window-segm.patch
Type: application/octet-stream
Size: 3182 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160827/281eb87c/attachment.obj>


More information about the ffmpeg-devel mailing list