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

dsp_module

#21

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


#22

any news?


#23

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


#24

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:


#25

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.