Re: [FFmpeg-devel] [PATCH] lsws/swscale.h: introduce sws_get_gaussian_vec

2023-11-04 Thread Stefano Sabatini
On date Wednesday 2023-09-06 13:13:25 +0200, Anton Khirnov wrote: > Quoting Stefano Sabatini (2023-09-06 00:59:44) [...] > A simple rename is a trivial API change. Almost everything else is not. > > > and therefore should not be committed. > > Yes, the baseline for every API change is that it is

Re: [FFmpeg-devel] [PATCH] lsws/swscale.h: introduce sws_get_gaussian_vec

2023-09-06 Thread Anton Khirnov
Quoting Stefano Sabatini (2023-09-06 00:59:44) > > > > As I already said above - function parameter names in a prototype are > > > > purely cosmetic and have no effect on anything besides doxygen. You can > > > > change them at will and even remove them entirely without breaking API > > > > or ABI.

Re: [FFmpeg-devel] [PATCH] lsws/swscale.h: introduce sws_get_gaussian_vec

2023-09-05 Thread Stefano Sabatini
On date Tuesday 2023-09-05 13:19:00 +0200, Anton Khirnov wrote: > Quoting Stefano Sabatini (2023-09-01 20:28:33) > > On date Friday 2023-09-01 17:50:56 +0200, Anton Khirnov wrote: > > > Quoting Stefano Sabatini (2023-08-31 17:06:06) > > > > On date Saturday 2023-08-26 17:15:36 +0200, Anton Khirnov

Re: [FFmpeg-devel] [PATCH] lsws/swscale.h: introduce sws_get_gaussian_vec

2023-09-05 Thread Anton Khirnov
Quoting Stefano Sabatini (2023-09-01 20:28:33) > On date Friday 2023-09-01 17:50:56 +0200, Anton Khirnov wrote: > > Quoting Stefano Sabatini (2023-08-31 17:06:06) > > > On date Saturday 2023-08-26 17:15:36 +0200, Anton Khirnov wrote: > > > > Quoting Stefano Sabatini (2023-08-26 14:23:28) > > > > >

Re: [FFmpeg-devel] [PATCH] lsws/swscale.h: introduce sws_get_gaussian_vec

2023-09-03 Thread Michael Niedermayer
On Sun, Sep 03, 2023 at 02:25:07AM +0200, Stefano Sabatini wrote: > On date Saturday 2023-09-02 22:07:53 +0200, Michael Niedermayer wrote: > > On Fri, Sep 01, 2023 at 08:38:26PM +0200, Stefano Sabatini wrote: > > > On date Friday 2023-09-01 18:54:40 +0200, Michael Niedermayer wrote: > > > > On Thu,

Re: [FFmpeg-devel] [PATCH] lsws/swscale.h: introduce sws_get_gaussian_vec

2023-09-02 Thread Stefano Sabatini
On date Saturday 2023-09-02 22:07:53 +0200, Michael Niedermayer wrote: > On Fri, Sep 01, 2023 at 08:38:26PM +0200, Stefano Sabatini wrote: > > On date Friday 2023-09-01 18:54:40 +0200, Michael Niedermayer wrote: > > > On Thu, Aug 31, 2023 at 07:16:20PM +0200, Stefano Sabatini wrote: > > [...] > > >

Re: [FFmpeg-devel] [PATCH] lsws/swscale.h: introduce sws_get_gaussian_vec

2023-09-02 Thread Michael Niedermayer
On Fri, Sep 01, 2023 at 08:38:26PM +0200, Stefano Sabatini wrote: > On date Friday 2023-09-01 18:54:40 +0200, Michael Niedermayer wrote: > > On Thu, Aug 31, 2023 at 07:16:20PM +0200, Stefano Sabatini wrote: > [...] > > > +/** > > > + * Compute and return a normalized Gaussian vector. > > > + * > >

Re: [FFmpeg-devel] [PATCH] lsws/swscale.h: introduce sws_get_gaussian_vec

2023-09-01 Thread Stefano Sabatini
On date Friday 2023-09-01 18:54:40 +0200, Michael Niedermayer wrote: > On Thu, Aug 31, 2023 at 07:16:20PM +0200, Stefano Sabatini wrote: [...] > > +/** > > + * Compute and return a normalized Gaussian vector. > > + * > > + * @param vecp: pointer where the computed vector is put in case of > > + *

Re: [FFmpeg-devel] [PATCH] lsws/swscale.h: introduce sws_get_gaussian_vec

2023-09-01 Thread Stefano Sabatini
On date Friday 2023-09-01 17:50:56 +0200, Anton Khirnov wrote: > Quoting Stefano Sabatini (2023-08-31 17:06:06) > > On date Saturday 2023-08-26 17:15:36 +0200, Anton Khirnov wrote: > > > Quoting Stefano Sabatini (2023-08-26 14:23:28) > > > > Use in place of sws_getGaussianVec. > > > > > > > > The

Re: [FFmpeg-devel] [PATCH] lsws/swscale.h: introduce sws_get_gaussian_vec

2023-09-01 Thread Michael Niedermayer
On Thu, Aug 31, 2023 at 07:16:20PM +0200, Stefano Sabatini wrote: > On date Thursday 2023-08-31 18:51:52 +0200, Andreas Rheinhardt wrote: > > Stefano Sabatini: > > > +int sws_get_gaussian_vec(SwsVector **vecp, > > > + AVClass *log_ctx, > > > + double

Re: [FFmpeg-devel] [PATCH] lsws/swscale.h: introduce sws_get_gaussian_vec

2023-09-01 Thread Anton Khirnov
Quoting Stefano Sabatini (2023-08-31 17:06:06) > On date Saturday 2023-08-26 17:15:36 +0200, Anton Khirnov wrote: > > Quoting Stefano Sabatini (2023-08-26 14:23:28) > > > Use in place of sws_getGaussianVec. > > > > > > The new function enable better log handling, and provide better naming > > > fo

Re: [FFmpeg-devel] [PATCH] lsws/swscale.h: introduce sws_get_gaussian_vec

2023-08-31 Thread Stefano Sabatini
On date Thursday 2023-08-31 18:51:52 +0200, Andreas Rheinhardt wrote: > Stefano Sabatini: > > +int sws_get_gaussian_vec(SwsVector **vecp, > > + AVClass *log_ctx, > > + double standard_deviation, double quality); > > > > Seriously? A pointer to an A

Re: [FFmpeg-devel] [PATCH] lsws/swscale.h: introduce sws_get_gaussian_vec

2023-08-31 Thread Andreas Rheinhardt
Stefano Sabatini: > +int sws_get_gaussian_vec(SwsVector **vecp, > + AVClass *log_ctx, > + double standard_deviation, double quality); > Seriously? A pointer to an AVClass as log_ctx? It is actually AVClass** (the logcontext must have a pointer to a

Re: [FFmpeg-devel] [PATCH] lsws/swscale.h: introduce sws_get_gaussian_vec

2023-08-31 Thread Stefano Sabatini
On date Saturday 2023-08-26 17:15:27 +0200, Andreas Rheinhardt wrote: > Stefano Sabatini: > > Use in place of sws_getGaussianVec. > > > > The new function enable better log handling, and provide better naming > > Better log handling? Why? > > > for the variance variable, now named standard_devia

Re: [FFmpeg-devel] [PATCH] lsws/swscale.h: introduce sws_get_gaussian_vec

2023-08-31 Thread Stefano Sabatini
On date Saturday 2023-08-26 17:15:36 +0200, Anton Khirnov wrote: > Quoting Stefano Sabatini (2023-08-26 14:23:28) > > Use in place of sws_getGaussianVec. > > > > The new function enable better log handling, and provide better naming > > for the variance variable, now named standard_deviation to re

Re: [FFmpeg-devel] [PATCH] lsws/swscale.h: introduce sws_get_gaussian_vec

2023-08-26 Thread Anton Khirnov
Quoting Stefano Sabatini (2023-08-26 14:23:28) > Use in place of sws_getGaussianVec. > > The new function enable better log handling, and provide better naming > for the variance variable, now named standard_deviation to reflect the > meaning of the parameter. Logging to NULL does not seem like a

Re: [FFmpeg-devel] [PATCH] lsws/swscale.h: introduce sws_get_gaussian_vec

2023-08-26 Thread Andreas Rheinhardt
Stefano Sabatini: > Use in place of sws_getGaussianVec. > > The new function enable better log handling, and provide better naming Better log handling? Why? > for the variance variable, now named standard_deviation to reflect the > meaning of the parameter. > --- > doc/APIchanges |

[FFmpeg-devel] [PATCH] lsws/swscale.h: introduce sws_get_gaussian_vec

2023-08-26 Thread Stefano Sabatini
Use in place of sws_getGaussianVec. The new function enable better log handling, and provide better naming for the variance variable, now named standard_deviation to reflect the meaning of the parameter. --- doc/APIchanges | 3 +++ libswscale/swscale.h | 21 ++-