The switch statement in we_fl_unix.c in function WpeHandleFileManager() runs from 554 to 1435 (at the moment) and every case statement runs through! The break statements are all conditional!
It is unclear whether this is intentional, but for a switch statement of approx. 900 lines, I would say it is a problem!
For that reason I create this issue: I cannot solve these warnings of implicit fallthrough in a reasonable time, because it is impossible to determine whether the fallthrough is correct!!
Given time we should have a go at refactoring this monstrosity!!
The switch statement in
we_fl_unix.c
in functionWpeHandleFileManager()
runs from 554 to 1435 (at the moment) and everycase
statement runs through! Thebreak
statements are all conditional!It is unclear whether this is intentional, but for a switch statement of approx. 900 lines, I would say it is a problem!
For that reason I create this issue: I cannot solve these warnings of implicit fallthrough in a reasonable time, because it is impossible to determine whether the fallthrough is correct!!
Given time we should have a go at refactoring this monstrosity!!
P.S. I consider this code to be a major bug.