search

commaai / panda

code powering the comma.ai panda
MIT License
1.5k stars 744 forks source link

remove void return #1963

Closed dzid26 closed 1 month ago

dzid26 commented 1 month ago

It looks that's the only one

adeebshihadeh commented 1 month ago

Is there a cppcheck or compiler flag we can enable to catch this?

dzid26 commented 1 month ago

It would be -Wpedantic or some other ISO-C option which in turn ignores -std=gnu11 and throws bunch of other errors. I tried selectively suppressing them with #pragmas, but that didn't want to work. It would need to be investigated more, but so far I am pessimistic.

I haven't seen this check in cppcheck.