Messages in this thread | | | Date | Mon, 01 Sep 2008 16:29:34 +0100 | From | "Jan Beulich" <> | Subject | Re: [PATCH 5/5] debug: BUILD_BUG_ON: error on non-const expressions |
| |
>>> Boaz Harrosh <bharrosh@panasas.com> 01.09.08 17:00 >>> >What is broken with my BUILD_BUG_ON_ZERO(). I tried all tests and >it works fine. Do you have a test with unwanted results? >(Actually it's the original one I have not touched it).
That's the problem - it uses the same sizeof(char[]) approach, and hence has the same problems that you just try to fix for BUILD_BUG_ON().
>>>> Of course you could wrap the whole thing in ({}), >>> "do{}while(0)" is effectively an "{}" plus the added bonus >>> of demanding an ";" ;-) >> >> An expression likewise demands a terminating ; (or a continuation of the >> expression, i.e. by using an operator) >> > >I was not criticizing your approach, I was commenting on: >"{}" vs. do{}while(0)
I think we're having some mis-understanding here: What I'm concerned about is that you can't use statements where-ever expressions can be used (the opposite is true, because expressions are statements). My main concern is the limitation of its use inside macros, where one could try to use comma expressions to combine multiple BUILD_BUG_ON()-s.
Jan
| |