On Fri, Feb 16, 2018 at 08:35:11AM -0600, Josh Poimboeuf wrote: > On Fri, Feb 16, 2018 at 07:55:13PM +0530, Progyan Bhattacharya wrote: > > Replace range expressions with seperate individual cases, i.e. convert case > > 1...3: to case 1: case 2: case 3 > > Range expression within case statements are non-standard C code and can > > create issues over compiler and platform variety. > > > > While compiling with gcc 4.8 (RHEL) I encountered this error on range > > expression in case statements: > > error: range expressions in switch statements are non-standard > > [-Werror=pedantic] > > > > Signed-off-by: Progyan Bhattacharya <progy...@acm.org> > > Hi Progyan, > > Thank you for the patch. > > I think this makes the code unnecessarily verbose and less readable. We > rely on many such GCC extensions, and we don't aim to comply with > standard C. And AFAIK, we don't use -Werror=pedantic in the kernel.
Agreed, it makes the code actively worse. Just don't use error=pedantic.