[FFmpeg-devel] [PATCH v2] avformat/hlsenc: improve the code readable of 1000000

[Steven Liu]

2017-01-23 23:18 GMT+08:00 wm4 <nfxjfg at googlemail.com>:

> On Mon, 23 Jan 2017 19:01:14 +0800
> Steven Liu <lq at chinaffmpeg.org> wrote:
>
> > Reviewed-by: Bodecs Bela <bodecsb at vivanet.hu>
> > Signed-off-by: Steven Liu <lq at chinaffmpeg.org>
> > ---
> >  libavformat/hlsenc.c | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
> > index 85d3955..0170b34 100644
> > --- a/libavformat/hlsenc.c
> > +++ b/libavformat/hlsenc.c
> > @@ -47,6 +47,7 @@ typedef enum {
> >
> >  #define KEYSIZE 16
> >  #define LINE_BUFFER_SIZE 1024
> > +#define HLS_MICROSECOND_UNIT   1000000
> >
> >  typedef struct HLSSegment {
> >      char filename[1024];
> > @@ -501,7 +502,7 @@ static int hls_append_segment(struct AVFormatContext
> *s, HLSContext *hls, double
> >                  return AVERROR(ENOMEM);
> >              }
> >              if (replace_int_data_in_filename(hls->avf->filename,
> sizeof(hls->avf->filename),
> > -                filename, 't',  (int64_t)round(1000000 * duration)) <
> 1) {
> > +                filename, 't',  (int64_t)round(duration *
> HLS_MICROSECOND_UNIT)) < 1) {
> >                  av_log(hls, AV_LOG_ERROR,
> >                         "Invalid second level segment filename template
> '%s', "
> >                          "you can try to remove
> second_level_segment_time flag\n",
>
> The commit message could be improved. It's barely English. (I'm not a
> native speaker either, so I have no suggestion.)
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

pushed