projectNe10 / Ne10

An open optimized software library project for the ARM® Architecture
Other
1.46k stars 408 forks source link

Missing break in case statement #266

Open edf-spire opened 3 years ago

edf-spire commented 3 years ago

Hi, if anybody is interested, this case statement is missing the break statement. The error was introduced several years ago:

https://github.com/projectNe10/Ne10/blob/1f059a764d0e1bc2481c0055c0e71538470baa83/modules/dsp/NE10_fft_generic_float32.c#L652

This refactor https://github.com/projectNe10/Ne10/commit/79c3d787302f8d74b9bcfe6545d487cdf1b101d9#diff-7bc71b336fad4d87a079da7976508f25395f0ca604c5e19b4be9919007a5e4a0 missed the opportunity to spot the misplaced break and instead removed it.