[FFmpeg-devel] [PATCH 1/2] doc/examples/filter_audio: use av_err2str

[Tristan Matthews]

On Thu, May 22, 2025 at 1:42 PM Marvin Scholz <epirat07 at gmail.com> wrote:
>
> On 22 May 2025, at 18:57, Tristan Matthews wrote:
>
> > ---
> >  doc/examples/filter_audio.c | 4 +---
> >  1 file changed, 1 insertion(+), 3 deletions(-)
> >
> > diff --git a/doc/examples/filter_audio.c b/doc/examples/filter_audio.c
> > index 8b237e2adf..02222f5914 100644
> > --- a/doc/examples/filter_audio.c
> > +++ b/doc/examples/filter_audio.c
> > @@ -270,7 +270,6 @@ int main(int argc, char *argv[])
> >      AVFilterGraph *graph;
> >      AVFilterContext *src, *sink;
> >      AVFrame *frame;
> > -    uint8_t errstr[1024];
> >      float duration;
> >      int err, nb_frames, i;
> >
> > @@ -354,7 +353,6 @@ int main(int argc, char *argv[])
> >      return 0;
> >
> >  fail:
> > -    av_strerror(err, errstr, sizeof(errstr));
> > -    fprintf(stderr, "%s\n", errstr);
> > +    fprintf(stderr, "%s\n", av_err2str(err));
> >      return 1;
> >  }
> > --
> > 2.45.2
>
> Patchset LGTM

Thanks, BTW thanks to your earlier patches that did the same, after
this there will no longer be any calls to av_strerror in the codebase
(aside from being called by av_err2str).

Best,
Tristan

>
> >
> > _______________________________________________
> > ffmpeg-devel mailing list
> > ffmpeg-devel at ffmpeg.org
> > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >
> > To unsubscribe, visit link above, or email
> > ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".