Why does the dsp-module uses unsigned types for sample/channel indexes?



Unsigned is fine, as long as the casts go away. But, at the very least, the code needs to be consistent in its usage of one or the other


any news?


Haven’t really had much time to think about it TBH.


Please use signed values. The C++ core guidelines advises signed, and once again, the fact that the STL uses unsigned is a design flaw that is acknowledged.
Lots of benefits:


I think all the C++ gurus say “use regular signed int” now days.

And we should still be cautious of code that “requires” static_cast. So, code should be revised to not require the cast, IMHO.