Fix compiler warning in MT6835.cpp

simplefoc / Arduino-FOC-drivers

Drivers and support code for SimpleFOC

https://docs.simplefoc.com/drivers_library

MIT License

146 stars 63 forks source link

Closed ystradmann closed 5 months ago

ystradmann commented 5 months ago

This change fixes a warning about an implicit overflow of MT6835_REG_CAL_STATUS.

unsigned conversion from 'int' to 'uint8_t' {aka 'unsigned char'} changes value from '275' to '19' [-Werror=overflow]

runger1101001 commented 5 months ago

Thanks for this! I think I have made the same change locally already also :-D

We only take PRs against the "dev" branch of the code, so I changed that. After the tests run I will merge it.

ystradmann commented 5 months ago

Perfect, thanks a lot!