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

James Almer jamrial at gmail.com
Mon Jan 23 16:14:00 EET 2017


On 1/23/2017 8:01 AM, Steven Liu 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

You could instead use AV_TIME_BASE. It's more in line with the rest of the
codebase.

>  
>  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",
> 



More information about the ffmpeg-devel mailing list