commaai / panda

code powering the comma.ai panda
MIT License
1.52k stars 763 forks source link

enable misra-config checks #1879

Closed adeebshihadeh closed 5 months ago

dzid26 commented 6 months ago

Did you notice -DSTM32F413xx & -DSTM32H725xx option triggered more rule violations. (10.1, 10.4, 11.8, 12.2 ) rules.txt Shouldn't -DSTM32F413xx & -DSTM32H725xx options be mainlined and each rule tackled separately again?

adeebshihadeh commented 6 months ago

Did you notice -DSTM32F413xx & -DSTM32H725xx option triggered more rule violations. (10.1, 10.4, 11.8, 12.2 ) rules.txt Shouldn't -DSTM32F413xx & -DSTM32H725xx options be mainlined and each rule tackled separately again?

I'm including those as part of enabling this.