Re: [FFmpeg-devel] [PATCH 1/2] avfilter: mark scale2ref as supporting dynamic sizes

2024-03-16 Thread Michael Niedermayer
On Wed, Mar 13, 2024 at 01:24:24PM +0100, Niklas Haas wrote:
> From: Niklas Haas 
> 
> Analogous to the "scale" filter, which this is practically identical
> with.
> ---
>  libavfilter/avfilter.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/libavfilter/avfilter.c b/libavfilter/avfilter.c
> index 831871de90b..dcad4d55292 100644
> --- a/libavfilter/avfilter.c
> +++ b/libavfilter/avfilter.c
> @@ -1027,7 +1027,8 @@ int ff_filter_frame(AVFilterLink *link, AVFrame *frame)
>  strcmp(link->dst->filter->name, "format") &&
>  strcmp(link->dst->filter->name, "idet") &&
>  strcmp(link->dst->filter->name, "null") &&
> -strcmp(link->dst->filter->name, "scale")) {
> +strcmp(link->dst->filter->name, "scale") &&
> +strcmp(link->dst->filter->name, "scale2ref")) {
>  av_assert1(frame->format== link->format);
>  av_assert1(frame->width == link->w);
>  av_assert1(frame->height== link->h);

LGTM

[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin


signature.asc
Description: PGP signature
___
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".


Re: [FFmpeg-devel] [PATCH 1/2] avfilter: mark scale2ref as supporting dynamic sizes

2024-03-13 Thread Gyan Doshi




On 2024-03-13 05:54 pm, Niklas Haas wrote:

From: Niklas Haas 

Analogous to the "scale" filter, which this is practically identical
with.
---
  libavfilter/avfilter.c | 3 ++-
  1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/libavfilter/avfilter.c b/libavfilter/avfilter.c
index 831871de90b..dcad4d55292 100644
--- a/libavfilter/avfilter.c
+++ b/libavfilter/avfilter.c
@@ -1027,7 +1027,8 @@ int ff_filter_frame(AVFilterLink *link, AVFrame *frame)
  strcmp(link->dst->filter->name, "format") &&
  strcmp(link->dst->filter->name, "idet") &&
  strcmp(link->dst->filter->name, "null") &&
-strcmp(link->dst->filter->name, "scale")) {
+strcmp(link->dst->filter->name, "scale") &&
+strcmp(link->dst->filter->name, "scale2ref")) {
  av_assert1(frame->format== link->format);
  av_assert1(frame->width == link->w);
  av_assert1(frame->height== link->h);


LGTM.

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-requ...@ffmpeg.org with subject "unsubscribe".


[FFmpeg-devel] [PATCH 1/2] avfilter: mark scale2ref as supporting dynamic sizes

2024-03-13 Thread Niklas Haas
From: Niklas Haas 

Analogous to the "scale" filter, which this is practically identical
with.
---
 libavfilter/avfilter.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/libavfilter/avfilter.c b/libavfilter/avfilter.c
index 831871de90b..dcad4d55292 100644
--- a/libavfilter/avfilter.c
+++ b/libavfilter/avfilter.c
@@ -1027,7 +1027,8 @@ int ff_filter_frame(AVFilterLink *link, AVFrame *frame)
 strcmp(link->dst->filter->name, "format") &&
 strcmp(link->dst->filter->name, "idet") &&
 strcmp(link->dst->filter->name, "null") &&
-strcmp(link->dst->filter->name, "scale")) {
+strcmp(link->dst->filter->name, "scale") &&
+strcmp(link->dst->filter->name, "scale2ref")) {
 av_assert1(frame->format== link->format);
 av_assert1(frame->width == link->w);
 av_assert1(frame->height== link->h);
-- 
2.44.0

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