Error compiling for board Arduino Leonardo: a function-definition is not allowed here before '{' token

[akakikuumeri]

Compiling the velocity_control.ino example and all other sketches I have tried, using v. 2.2.1, targeting Arduino Leonardo in Arduino IDE on Windows exit with the following errors:

C:\Users\*****\Documents\Arduino\libraries\Simple_FOC\src\drivers\hardware_specific\atmega32u4_mcu.cpp: In function 'int _configure6PWM(long int, float, int, int, int, int, int, int)':
C:\Users\*****\Documents\Arduino\libraries\Simple_FOC\src\drivers\hardware_specific\atmega32u4_mcu.cpp:151:1: error: a function-definition is not allowed here before '{' token
 {
 ^
C:\Users\*****\Documents\Arduino\libraries\Simple_FOC\src\drivers\hardware_specific\atmega32u4_mcu.cpp:166:151: error: a function-definition is not allowed here before '{' token
 void _writeDutyCycle6PWM(float dc_a,  float dc_b, float dc_c, float dead_zone, int pinA_h, int pinA_l, int pinB_h, int pinB_l, int pinC_h, int pinC_l){
                                                                                                                                                       ^
C:\Users\***\Documents\Arduino\libraries\Simple_FOC\src\drivers\hardware_specific\atmega32u4_mcu.cpp:170:1: error: expected '}' at end of input
 }
 ^
exit status 1
Error compiling for board Arduino Leonardo.

Further, I also get multiple warnings like

C:\Users\*****\Documents\Arduino\libraries\Simple_FOC\src/communication/Commander.h:210:68: warning: ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]
     void target(FOCMotor* motor, char* user_cmd, char* separator = " ");

on both Leonardo but also on Arduino MEGA, but there they don't prevent me from compiling, so I am not sure if they are related to the error.

[askuric]

Hey @akakikuumeri, it seems like there is a forgotten bracket :/

In the file atmega32u4_mcu.h on the line 147.

Arduino-FOC/src/drivers/hardware_specific/atmega32u4_mcu.cpp

Line 137 in 47e1a7b

int _configure6PWM(long pwm_frequency, float dead_zone, const int pinA_h, const int pinA_l, const int pinB_h, const int pinB_l, const int pinC_h, const int pinC_l) {

The code for the Leonardo is not much tested, please do not hesitate to post issues if you'll have ,more of them. Its going to help us improve our support!

Btw, the warning is ok, not a big deal. It will be removed in the next release.

[akakikuumeri]

Thanks a lot! I should've been able to fix that myself, but I'm only getting used to the project. Thank you for your help.

I'm trying to get an Arduino Pro Micro to work with some AS5600 sensors and a stepper motor. I'll post issues if I run into more.

[askuric]

Fixed in the dev branch, by richard few months ago.
The bug will be removed in the next release v2.2.2