grblHAL / core

grblHAL core code and master Wiki
Other
323 stars 84 forks source link

Minor: Compiling recent versions in Arduino IDE throws declaration warnings... #108

Closed NeilMarley closed 2 years ago

NeilMarley commented 2 years ago

Is seems that a few new variables/functions (Which I suspect are only relevant in some options/plug-ins) are throwing-up warnings on compilation that they declared but unused....

Compiler output:

......Documents\Arduino\libraries\grblHAL_Teensy4\src\driver.c: In function 'driver_delay_ms': C:\Users\Neil\Documents\Arduino\libraries\grblHAL_Teensy4\src\driver.c:574:39: warning: implicit declaration of function 'state_get' [-Wimplicit-function-declaration] grbl.on_execute_delay(state_get()); ^ In file included from .....Documents\Arduino\libraries\grblHAL_Teensy4\src\driver.c:73:0: ....Documents\Arduino\libraries\grblHAL_Teensy4\src\driver.c: At top level: ....Documents\Arduino\libraries\grblHAL_Teensy4\src\grbl/motor_pins.h:696:13: warning: 'motor_iterator' defined but not used [-Wunused-function] static void motor_iterator (motor_iterator_callback_ptr callback) ^ ....Documents\Arduino\libraries\grblHAL_Teensy4\src\ioports.c:37:42: warning: 'analog_n_in' defined but not used [-Wunused-variable] static uint_fast8_t aux_n_in, aux_n_out, analog_n_in, in_map = NULL, out_map = NULL;;

terjeio commented 2 years ago

Thanks, I'll fix this in the next commit.