[alsa-devel] cp: upmix: code formatting and cleanup, generalize format

GitHub pull_request - opened github at alsa-project.org
Wed Nov 20 17:45:46 CET 2019


alsa-project/alsa-plugins pull request #2 was opened from bku-sue:

The main purpose of this change is to be able to change the format to e.g. S32 instead of S16 by changing only two lines on the top of the source. So a new symbol `UPMIX_PCM_FORMAT` is added to hide the actual format, and a special type `upmix_sample_t` marks the sample type, instead of fixed `short`.

I fixed the `average_copy` function as well, which was actually expecting the `nchns` parameter to be 2, otherwise its local arrays would be overindexed.

The rest is just better naming for `i` as it denotes the channel index in every case, and applying consistent whitespace.

The behavior of the plugin remains identical though.

Request URL   : https://github.com/alsa-project/alsa-plugins/pull/2
Patch URL     : https://github.com/alsa-project/alsa-plugins/pull/2.patch
Repository URL: https://github.com/alsa-project/alsa-plugins


More information about the Alsa-devel mailing list