Messages in this thread | | | Date | Fri, 16 Feb 2018 17:47:22 +0100 | From | Peter Zijlstra <> | Subject | Re: [PATCH] objtool/x86: Replace Non-standard Range Expression in Case |
| |
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 <progyanb@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.
| |