[FFmpeg-devel] [PATCH] doc/examples/qsv_transcode: EINVAL is more appropriate and ENAVAIL will fail build with visual studio

Xiang, Haihao haihao.xiang at intel.com
Thu Dec 7 09:36:55 EET 2023


On Do, 2023-12-07 at 06:44 +0000, hung kuishing wrote:
> Signed-off-by: clarkh <hungkuishing at outlook.com>
> ---
>  doc/examples/qsv_transcode.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/doc/examples/qsv_transcode.c b/doc/examples/qsv_transcode.c
> index 48128b200c..972126800b 100644
> --- a/doc/examples/qsv_transcode.c
> +++ b/doc/examples/qsv_transcode.c
> @@ -62,10 +62,10 @@ static int str_to_dict(char* optstr, AVDictionary **opt)
>          return 0;
>      key = strtok(optstr, " ");
>      if (key == NULL)
> -        return AVERROR(ENAVAIL);
> +        return AVERROR(EINVAL);
>      value = strtok(NULL, " ");
>      if (value == NULL)
> -        return AVERROR(ENAVAIL);
> +        return AVERROR(EINVAL);
>      av_dict_set(opt, key, value, 0);
>      do {
>          key = strtok(NULL, " ");
> @@ -73,7 +73,7 @@ static int str_to_dict(char* optstr, AVDictionary **opt)
>              return 0;
>          value = strtok(NULL, " ");
>          if (value == NULL)
> -            return AVERROR(ENAVAIL);
> +            return AVERROR(EINVAL);
>          av_dict_set(opt, key, value, 0);
>      } while(key != NULL);
>      return 0;
> @@ -181,7 +181,7 @@ static int open_input_file(char *filename)
>          break;
>      default:
>          fprintf(stderr, "Codec is not supportted by qsv\n");
> -        return AVERROR(ENAVAIL);
> +        return AVERROR(EINVAL);
>      }
>  
>      if (!(decoder_ctx = avcodec_alloc_context3(decoder)))

LGTM, thanks for fixing the issue.

BRs
Haihao



More information about the ffmpeg-devel mailing list