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

Bodecs Bela bodecsb at vivanet.hu
Mon Jan 23 16:08:00 EET 2017



2017.01.23. 12:01 keltezéssel, Steven Liu írta:
> 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",
I think it is good.

bb



More information about the ffmpeg-devel mailing list