Released libsidplayfp-2.9.0
As suggested the proper way to silence the warning is adding an assert. This however would require some machinery to define NDEBUG for standard builds in resid (and while at it maybe rename the configure.in into a more modern configure.ac)
Bug filed: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116315
BTW, I see VICE now requires c++11, is that true for resid too? It would be even better replacing that enum values with constexpr definitions maybe.
Hmmm, this trivial patch silence the warning: --- a/src/resid/filter8580new.cc +++ b/src/resid/filter8580new.cc @@ -300,6 +300,7 @@ Filter::Filter() // double_point scaled_voltage[fi.opamp_voltage_size]; double_point scaled_voltage[50]; + const int opamp_voltage_last = fi.opamp_voltage_size - 1; for (int i = 0; i < fi.opamp_voltage_size; i++) { // The target output range is 16 bits, in order to fit in an unsigned // short. @@ -329,7 +330,7 @@ Filter::Filter() scaled_voltage[fi.opamp_voltage_size...
resid: fix deprecated-enum-float-conversion warnings
Just for the record I can reproduce this with -O3 but not with -O2 (gcc-14.2)
Released libsidplayfp-2.8.0