Re: [FFmpeg-devel] [PATCH 3/4] avformat/hlsenc: Fix name of flag in error message

2023-10-27 Thread Steven Liu
Dave Johansen  于2023年10月27日周五 12:00写道:
>
> ---
>  libavformat/hlsenc.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
> index 24a0304f78..93c47b631b 100644
> --- a/libavformat/hlsenc.c
> +++ b/libavformat/hlsenc.c
> @@ -1013,7 +1013,7 @@ static int sls_flags_filename_process(struct 
> AVFormatContext *s, HLSContext *hls
>   '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",
> +   "you can try to remove second_level_segment_duration 
> flag\n",
> vs->avf->url);
>  av_freep(&filename);
>  return AVERROR(EINVAL);
> @@ -1106,7 +1106,7 @@ static int 
> sls_flag_use_localtime_filename(AVFormatContext *oc, HLSContext *c, V
>  char *filename = NULL;
>  if (replace_int_data_in_filename(&filename, oc->url, 't', 0) < 
> 1) {
>  av_log(c, AV_LOG_ERROR, "Invalid second level segment 
> filename template '%s', "
> -"you can try to remove second_level_segment_time 
> flag\n",
> +"you can try to remove second_level_segment_duration 
> flag\n",
> oc->url);
>  av_freep(&filename);
>  return AVERROR(EINVAL);
> --
> 2.39.2 (Apple Git-143)
>
> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

LGTM
Good catch


Thanks
Steven
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".


[FFmpeg-devel] [PATCH 3/4] avformat/hlsenc: Fix name of flag in error message

2023-10-26 Thread Dave Johansen
---
 libavformat/hlsenc.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
index 24a0304f78..93c47b631b 100644
--- a/libavformat/hlsenc.c
+++ b/libavformat/hlsenc.c
@@ -1013,7 +1013,7 @@ static int sls_flags_filename_process(struct 
AVFormatContext *s, HLSContext *hls
  '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",
+   "you can try to remove second_level_segment_duration 
flag\n",
vs->avf->url);
 av_freep(&filename);
 return AVERROR(EINVAL);
@@ -1106,7 +1106,7 @@ static int 
sls_flag_use_localtime_filename(AVFormatContext *oc, HLSContext *c, V
 char *filename = NULL;
 if (replace_int_data_in_filename(&filename, oc->url, 't', 0) < 1) {
 av_log(c, AV_LOG_ERROR, "Invalid second level segment filename 
template '%s', "
-"you can try to remove second_level_segment_time 
flag\n",
+"you can try to remove second_level_segment_duration 
flag\n",
oc->url);
 av_freep(&filename);
 return AVERROR(EINVAL);
-- 
2.39.2 (Apple Git-143)

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".