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 > > > for the variance variable, now named standard_deviation to reflect the > > > meaning of the parameter. > > > > > Logging to NULL does not seem like an improvement to me. > > Adding the log_ctx. > > > Renaming a function parameter does not require an API break. > > The main point was improving the naming of the variable, but while at > it I'm also adding the logging context and providing a return code to > specify an error failure, and moving to snake_case convention which is > the one used by the new API additions.
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. The other reasons do not strike me as strong enough to warrant an API break. -- Anton Khirnov _______________________________________________ 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".