Messages in this thread |  | | Date | Mon, 19 May 2014 12:08:39 +0300 | | From | Dan Carpenter <> | | Subject | Re: [PATCH v2 05/06] staging: crypto: skein: rename macros |
| |
On Mon, May 19, 2014 at 12:09:58PM +0400, Anton Saraev wrote: > +/* ignore all asserts, for performance */ > +#define skein_assert_ret(x, ret_code) > +#define skein_assert(x)
Not related to this patch, but defining away asserts like this is a bad idea. What if they have side affects like:
skein_assert(foo++ == bar);
regards, dan carpenter
|  |