[FFmpeg-devel] [PATCH] ffmpeg_demux: don't print input error for AVERROR_EXIT

[FFmpeg-devel] [PATCH] ffmpeg_demux: don't print input error for AVERROR_EXIT

[Gyan Doshi]

This error code is for expected exits, like after listing options for a
device.
---
 fftools/ffmpeg_demux.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/fftools/ffmpeg_demux.c b/fftools/ffmpeg_demux.c
index 0fd095d8ae..5ff75eed1a 100644
--- a/fftools/ffmpeg_demux.c
+++ b/fftools/ffmpeg_demux.c
@@ -1726,8 +1726,9 @@ int ifile_open(const OptionsContext *o, const char *filename, Scheduler *sch)
     /* open the input file with generic avformat function */
     err = avformat_open_input(&ic, filename, file_iformat, &o->g->format_opts);
     if (err < 0) {
-        av_log(d, AV_LOG_ERROR,
-               "Error opening input: %s\n", av_err2str(err));
+        if (err != AVERROR_EXIT)
+            av_log(d, AV_LOG_ERROR,
+                   "Error opening input: %s\n", av_err2str(err));
         if (err == AVERROR_PROTOCOL_NOT_FOUND)
             av_log(d, AV_LOG_ERROR, "Did you mean file:%s?\n", filename);
         return err;
-- 
2.46.1

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

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

Re: [FFmpeg-devel] [PATCH] ffmpeg_demux: don't print input error for AVERROR_EXIT

[Gyan Doshi]

On 2025-01-27 10:48 am, Gyan Doshi wrote:
> This error code is for expected exits, like after listing options for a
> device.

Plan to push tomorrow.

Regards,
Gyan


> ---
>   fftools/ffmpeg_demux.c | 5 +++--
>   1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/fftools/ffmpeg_demux.c b/fftools/ffmpeg_demux.c
> index 0fd095d8ae..5ff75eed1a 100644
> --- a/fftools/ffmpeg_demux.c
> +++ b/fftools/ffmpeg_demux.c
> @@ -1726,8 +1726,9 @@ int ifile_open(const OptionsContext *o, const char *filename, Scheduler *sch)
>       /* open the input file with generic avformat function */
>       err = avformat_open_input(&ic, filename, file_iformat, &o->g->format_opts);
>       if (err < 0) {
> -        av_log(d, AV_LOG_ERROR,
> -               "Error opening input: %s\n", av_err2str(err));
> +        if (err != AVERROR_EXIT)
> +            av_log(d, AV_LOG_ERROR,
> +                   "Error opening input: %s\n", av_err2str(err));
>           if (err == AVERROR_PROTOCOL_NOT_FOUND)
>               av_log(d, AV_LOG_ERROR, "Did you mean file:%s?\n", filename);
>           return err;

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

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

Re: [FFmpeg-devel] [PATCH] ffmpeg_demux: don't print input error for AVERROR_EXIT

[Gyan Doshi]

On 2025-01-29 10:24 am, Gyan Doshi wrote:
>
>
> On 2025-01-27 10:48 am, Gyan Doshi wrote:
>> This error code is for expected exits, like after listing options for a
>> device.
>
> Plan to push tomorrow.

Pushed as 1911a6ec26236588f72413c2b9bfe54a97a6a145

Regards,
Gyan

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

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