[FFmpeg-devel] [PATCH 1/2] swresample/rematrix: split filling the matrix array into its own function

[Michael Niedermayer]

On Sun, Feb 02, 2025 at 06:46:41PM -0300, James Almer wrote:
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
>  libswresample/rematrix.c | 202 ++++++++++++++++++++-------------------
>  1 file changed, 106 insertions(+), 96 deletions(-)
> 
> diff --git a/libswresample/rematrix.c b/libswresample/rematrix.c
> index b9bf4dcac0..8b6e8ae1c5 100644
> --- a/libswresample/rematrix.c
> +++ b/libswresample/rematrix.c
> @@ -124,85 +124,28 @@ static int sane_layout(AVChannelLayout *ch_layout) {
>      return 1;
>  }
>  
> -av_cold int swr_build_matrix2(const AVChannelLayout *in_layout, const AVChannelLayout *out_layout,
> -                              double center_mix_level, double surround_mix_level,
> -                              double lfe_mix_level, double maxval,
> -                              double rematrix_volume, double *matrix_param,
> -                              ptrdiff_t stride, enum AVMatrixEncoding matrix_encoding, void *log_context)
> +static void build_matrix(const AVChannelLayout *in_ch_layout, const AVChannelLayout *out_ch_layout,
> +                         double center_mix_level, double surround_mix_level,
> +                         double lfe_mix_level, double maxval, double rematrix_volume, double *matrix_param,
> +                         ptrdiff_t stride, enum AVMatrixEncoding matrix_encoding)

Please document new functions, even if its just refering
to swr_build_matrix2() docs

thx

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

Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20250206/bd91846a/attachment.sig>