Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
* [FFmpeg-devel] [PATCH] avfilter/vf_colorspace: fix message for missing/incorrect colorspace
@ 2023-05-30 10:29 Sami Boukortt
  2023-05-30 13:06 ` Ronald S. Bultje
  0 siblings, 1 reply; 2+ messages in thread
From: Sami Boukortt @ 2023-05-30 10:29 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Sami Boukortt

---
 libavfilter/vf_colorspace.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/libavfilter/vf_colorspace.c b/libavfilter/vf_colorspace.c
index 21916b7b7a..852dc11c6c 100644
--- a/libavfilter/vf_colorspace.c
+++ b/libavfilter/vf_colorspace.c
@@ -572,15 +572,15 @@ static int create_filtergraph(AVFilterContext *ctx,
             if (s->out_csp == AVCOL_SPC_UNSPECIFIED) {
                 if (s->user_all == CS_UNSPECIFIED) {
                     av_log(ctx, AV_LOG_ERROR,
-                           "Please specify output transfer characteristics\n");
+                           "Please specify output colorspace\n");
                 } else {
                     av_log(ctx, AV_LOG_ERROR,
                            "Unsupported output color property %d\n", s->user_all);
                 }
             } else {
                 av_log(ctx, AV_LOG_ERROR,
-                       "Unsupported output transfer characteristics %d (%s)\n",
-                       s->out_csp, av_color_space_name(s->out_csp));
+                       "Unsupported output colorspace %d (%s)\n", s->out_csp,
+                       av_color_space_name(s->out_csp));
             }
             return AVERROR(EINVAL);
         }
-- 
2.41.0.rc0.172.g3f132b7071-goog

_______________________________________________
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".

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [FFmpeg-devel] [PATCH] avfilter/vf_colorspace: fix message for missing/incorrect colorspace
  2023-05-30 10:29 [FFmpeg-devel] [PATCH] avfilter/vf_colorspace: fix message for missing/incorrect colorspace Sami Boukortt
@ 2023-05-30 13:06 ` Ronald S. Bultje
  0 siblings, 0 replies; 2+ messages in thread
From: Ronald S. Bultje @ 2023-05-30 13:06 UTC (permalink / raw)
  To: FFmpeg development discussions and patches; +Cc: Sami Boukortt

Hi,

On Tue, May 30, 2023 at 6:29 AM Sami Boukortt <
sboukortt-at-google.com@ffmpeg.org> wrote:

> ---
>  libavfilter/vf_colorspace.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/libavfilter/vf_colorspace.c b/libavfilter/vf_colorspace.c
> index 21916b7b7a..852dc11c6c 100644
> --- a/libavfilter/vf_colorspace.c
> +++ b/libavfilter/vf_colorspace.c
> @@ -572,15 +572,15 @@ static int create_filtergraph(AVFilterContext *ctx,
>              if (s->out_csp == AVCOL_SPC_UNSPECIFIED) {
>                  if (s->user_all == CS_UNSPECIFIED) {
>                      av_log(ctx, AV_LOG_ERROR,
> -                           "Please specify output transfer
> characteristics\n");
> +                           "Please specify output colorspace\n");
>                  } else {
>                      av_log(ctx, AV_LOG_ERROR,
>                             "Unsupported output color property %d\n",
> s->user_all);
>                  }
>              } else {
>                  av_log(ctx, AV_LOG_ERROR,
> -                       "Unsupported output transfer characteristics %d
> (%s)\n",
> -                       s->out_csp, av_color_space_name(s->out_csp));
> +                       "Unsupported output colorspace %d (%s)\n",
> s->out_csp,
> +                       av_color_space_name(s->out_csp));
>              }
>              return AVERROR(EINVAL);
>          }
> --
> 2.41.0.rc0.172.g3f132b7071-goog
>

Good catch, thanks. Merged.

Ronald
_______________________________________________
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".

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-05-30 13:06 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-05-30 10:29 [FFmpeg-devel] [PATCH] avfilter/vf_colorspace: fix message for missing/incorrect colorspace Sami Boukortt
2023-05-30 13:06 ` Ronald S. Bultje

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git