SNEX weirdness or me ?
-
@Christoph-Hart So in the end I was doing something silly that didn't need a fix if I had followed the conventional approach of C++
-
@ustk yes, but it's the job of the compiler to tell you that you're doing something stupid and the actual effect it had on your class was totally unpredictable, so it's definitely better now than before :)
-
@Christoph-Hart Hmmm... So now it complains on the index type in the process template
private: using IndexType = index::clamped<MAX_NUM_CHANNELS, false>; IndexType idx;
But it is impossible to stick it as member parameter or declare it as constant, am I right?
So what would be the procedure then?
Sorry for the C++ crash course I still need... -
@Christoph-Hart both process and idx are in the same template so this shouldn't be an issue if I am not mistaken
-
@ustk Boing boing
-
@Christoph-Hart Even doing this throws the error:
It seems
index
is considered to be outside -
@ustk ah yeah actually I caught a similar error with the index classes in my test suite when I implemented the fix but it seems that the test suite didn‘t cover index usage within a class.
Back to the SNEX compiler internals I guess…
-
Alright, please try again, now it should not mess with the index templates anymore.
-
@Christoph-Hart Thanks! Building now...
-
@Christoph-Hart Working smoothly! Thanks a lot for the fix!