search

ARM-software / CMSIS-DSP

CMSIS-DSP embedded compute library for Cortex-M and Cortex-A
https://arm-software.github.io/CMSIS-DSP
Apache License 2.0
459 stars 123 forks source link

const-qualify table of Y values in interpolation #93

Closed escherstair closed 1 year ago

escherstair commented 1 year ago

Interpolation functions use Instance structures as this one https://github.com/ARM-software/CMSIS-DSP/blob/d7e4dea51ad19724adcd2c9f82eb42af12e0f5fc/Include/dsp/interpolation_functions.h#L50-L59

The pYData field points to the table of Y values, that must not (and is not) modified by the interpolation function. As done for other functions, I suggest to const-qualify the field, changing to

          const float32_t *pYData;          /**< pointer to the table of Y values */

And so on, wherever is necessary.

christophe0606 commented 1 year ago

@escherstair Thank you for the feedback. I'll do the change as soon as I have finished the tasks I am working on currently.

christophe0606 commented 1 year ago

Done in latest commit.