[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 14:15:10 EEST 2016
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
-------------- 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: 3169 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160827/11573ccc/attachment.obj>
More information about the ffmpeg-devel
mailing list