Thanks and a few observations

First off, I’d like to thank all the people who developed this software and the excellent web site.

I’ve been using the software for several weeks with much success.

One thing that I found somewhat disconcerting, though, is the number of warnings that I get when compiling with “–warnings all”. I like to use this for finding errors in my own code, but the sheer volume of warnings I get from the library itself. I can turn them off but then I don’t get any compiler help for my own code.

For example here’s a condensed list of warnings I get when compiling one of my projects.

Is there any hope that some of these could be fixed a future release?

Thanks again for this project and best regards,

--Derek

===
src/sensors/MagneticSensorSPI.h:38:5: warning: "/*" within comment [-Wcomment]
src/common/base_classes/Sensor.h:127:22: warning: 'virtual void Sensor::init()' was hidden [-Woverloaded-virtual=]
src/common/base_classes/Sensor.h:127:22: warning: 'virtual void Sensor::init()' was hidden [-Woverloaded-virtual=]
src/common/base_classes/CurrentSense.cpp:628:11: warning: unused variable 'ca' [-Wunused-variable]
src/common/base_classes/CurrentSense.cpp:701:11: warning: unused variable 'cb' [-Wunused-variable]
src/common/base_classes/FOCMotor.cpp:68:26: warning: arithmetic between enumeration type 'Direction' and floating-point type 'float' is deprecated [-Wdeprecated-enum-float-conversion]
src/common/base_classes/FOCMotor.cpp:74:26: warning: arithmetic between enumeration type 'Direction' and floating-point type 'float' is deprecated [-Wdeprecated-enum-float-conversion]
src/common/base_classes/FOCMotor.cpp:502:5: warning: this statement may fall through [-Wimplicit-fallthrough=]
src/common/base_classes/FOCMotor.cpp:504:5: warning: this statement may fall through [-Wimplicit-fallthrough=]
src/common/base_classes/FOCMotor.cpp:512:5: warning: this statement may fall through [-Wimplicit-fallthrough=]
src/communication/Commander.cpp:579:9: warning: enumeration value 'custom' not handled in switch [-Wswitch]
src/current_sense/hardware_specific/esp32/esp32_mcpwm_mcu.cpp:177:44: warning: missing initializer for member 'mcpwm_comparator_config_t::flags' [-Wmissing-field-initializers]
src/current_sense/hardware_specific/esp32/esp32_mcpwm_mcu.cpp:226:5: warning: missing initializer for member 'mcpwm_timer_event_callbacks_t::on_empty' [-Wmissing-field-initializers]
src/current_sense/hardware_specific/esp32/esp32_mcpwm_mcu.cpp:226:5: warning: missing initializer for member 'mcpwm_timer_event_callbacks_t::on_stop' [-Wmissing-field-initializers]
src/drivers/hardware_specific/generic_mcu.cpp:17:3: warning: missing initializer for member 'GenericDriverParams::dead_zone' [-Wmissing-field-initializers]
src/drivers/hardware_specific/generic_mcu.cpp:29:3: warning: missing initializer for member 'GenericDriverParams::dead_zone' [-Wmissing-field-initializers]
src/drivers/hardware_specific/generic_mcu.cpp:41:3: warning: missing initializer for member 'GenericDriverParams::dead_zone' [-Wmissing-field-initializers]
src/drivers/hardware_specific/generic_mcu.cpp:54:3: warning: missing initializer for member 'GenericDriverParams::dead_zone' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_mcpwm_mcu.cpp:79:5: warning: missing initializer for member 'ESP32MCPWMDriverParams::timers' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_mcpwm_mcu.cpp:79:5: warning: missing initializer for member 'ESP32MCPWMDriverParams::oper' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_mcpwm_mcu.cpp:79:5: warning: missing initializer for member 'ESP32MCPWMDriverParams::comparator' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_mcpwm_mcu.cpp:79:5: warning: missing initializer for member 'ESP32MCPWMDriverParams::generator' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_mcpwm_mcu.cpp:79:5: warning: missing initializer for member 'ESP32MCPWMDriverParams::mcpwm_period' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_mcpwm_mcu.cpp:79:5: warning: missing initializer for member 'ESP32MCPWMDriverParams::dead_zone' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_mcpwm_mcu.cpp:153:5: warning: missing initializer for member 'ESP32MCPWMDriverParams::comparator' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_mcpwm_mcu.cpp:153:5: warning: missing initializer for member 'ESP32MCPWMDriverParams::generator' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_mcpwm_mcu.cpp:153:5: warning: missing initializer for member 'ESP32MCPWMDriverParams::mcpwm_period' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_mcpwm_mcu.cpp:153:5: warning: missing initializer for member 'ESP32MCPWMDriverParams::dead_zone' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.cpp:108:7: warning: extra tokens at end of #else directive [-Wendif-labels]
.arduino15/packages/esp32/tools/esp32-libs/3.3.8/include/esp_driver_mcpwm/include/driver/mcpwm_gen.h:137:61: warning: missing initializer for member 'mcpwm_gen_compare_event_action_t::direction' [-Wmissing-field-initializers]
.arduino15/packages/esp32/tools/esp32-libs/3.3.8/include/esp_driver_mcpwm/include/driver/mcpwm_gen.h:137:61: warning: missing initializer for member 'mcpwm_gen_compare_event_action_t::action' [-Wmissing-field-initializers]
.arduino15/packages/esp32/tools/esp32-libs/3.3.8/include/esp_driver_mcpwm/include/driver/mcpwm_gen.h:137:61: warning: missing initializer for member 'mcpwm_gen_compare_event_action_t::direction' [-Wmissing-field-initializers]
.arduino15/packages/esp32/tools/esp32-libs/3.3.8/include/esp_driver_mcpwm/include/driver/mcpwm_gen.h:137:61: warning: missing initializer for member 'mcpwm_gen_compare_event_action_t::action' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.cpp:288:3: warning: missing initializer for member 'ESP32MCPWMDriverParams::timers' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.cpp:288:3: warning: missing initializer for member 'ESP32MCPWMDriverParams::oper' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.cpp:288:3: warning: missing initializer for member 'ESP32MCPWMDriverParams::comparator' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.cpp:288:3: warning: missing initializer for member 'ESP32MCPWMDriverParams::generator' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.cpp:288:3: warning: missing initializer for member 'ESP32MCPWMDriverParams::mcpwm_period' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.cpp:288:3: warning: missing initializer for member 'ESP32MCPWMDriverParams::dead_zone' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.cpp:308:71: warning: missing initializer for member 'mcpwm_operator_config_t::intr_priority' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.cpp:308:71: warning: missing initializer for member 'mcpwm_operator_config_t::flags' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.cpp:323:51: warning: missing initializer for member 'mcpwm_comparator_config_t::flags' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.cpp:416:3: warning: missing initializer for member 'ESP32MCPWMDriverParams::timers' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.cpp:416:3: warning: missing initializer for member 'ESP32MCPWMDriverParams::oper' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.cpp:416:3: warning: missing initializer for member 'ESP32MCPWMDriverParams::comparator' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.cpp:416:3: warning: missing initializer for member 'ESP32MCPWMDriverParams::generator' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.cpp:416:3: warning: missing initializer for member 'ESP32MCPWMDriverParams::mcpwm_period' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.cpp:416:3: warning: missing initializer for member 'ESP32MCPWMDriverParams::dead_zone' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.cpp:457:71: warning: missing initializer for member 'mcpwm_operator_config_t::intr_priority' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.cpp:457:71: warning: missing initializer for member 'mcpwm_operator_config_t::flags' [-Wmissing-field-initializers]
src/drivers/hardware_specific/esp32/esp32_driver_mcpwm.cpp:474:51: warning: missing initializer for member 'mcpwm_comparator_config_t::flags' [-Wmissing-field-initializers]
src/sensors/Encoder.cpp:56:9: warning: '++' expression of 'volatile'-qualified type is deprecated [-Wvolatile]
src/sensors/Encoder.cpp:77:9: warning: '--' expression of 'volatile'-qualified type is deprecated [-Wvolatile]
src/sensors/MagneticSensorAnalog.cpp:16:6: warning: '*this.MagneticSensorAnalog::pullup' is used uninitialized [-Wuninitialized]
src/sensors/HallSensor.cpp:77:3: warning: '++' expression of 'volatile'-qualified type is deprecated [-Wvolatile]
src/sensors/HallSensor.cpp:137:22: warning: arithmetic between enumeration type 'Direction' and floating-point type 'float' is deprecated [-Wdeprecated-enum-float-conversion]
src/common/base_classes/Sensor.h:127:22: warning: 'virtual void Sensor::init()' was hidden [-Woverloaded-virtual=]
src/sensors/MagneticSensorSPI.h:38:5: warning: "/*" within comment [-Wcomment]
src/common/base_classes/Sensor.h:127:22: warning: 'virtual void Sensor::init()' was hidden [-Woverloaded-virtual=]

===