diff options
| author | Rusty Russell <rusty@rustcorp.com.au> | 2011-12-05 16:42:50 +1030 |
|---|---|---|
| committer | Rusty Russell <rusty@rustcorp.com.au> | 2011-12-05 16:42:50 +1030 |
| commit | 3b390b754eb8edd43678bdd080b06f102766683d (patch) | |
| tree | 6420770fc0d3b02ca685342efb47ff70e301e5e1 /lib/ccan/compiler | |
| parent | ab2c07b8f0dec6fd82884b7864a399201c3d2ce3 (diff) | |
| download | samba-3b390b754eb8edd43678bdd080b06f102766683d.tar.gz samba-3b390b754eb8edd43678bdd080b06f102766683d.tar.bz2 samba-3b390b754eb8edd43678bdd080b06f102766683d.zip | |
lib/ccan/compiler, ilog: IDEMPOTENT "idempotent does not mean what you think it means"
Actually, I don't even think it means that. But rename it to something
which is sane.
Thanks to David Gibson for reporting.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
(Imported from CCAN commit e764d0a27d2b6748ea7d343042ec7d6dda1f6aae)
Diffstat (limited to 'lib/ccan/compiler')
| -rw-r--r-- | lib/ccan/compiler/_info | 4 | ||||
| -rw-r--r-- | lib/ccan/compiler/compiler.h | 8 |
2 files changed, 6 insertions, 6 deletions
diff --git a/lib/ccan/compiler/_info b/lib/ccan/compiler/_info index e42ebb0fb3..9f39453973 100644 --- a/lib/ccan/compiler/_info +++ b/lib/ccan/compiler/_info @@ -10,7 +10,7 @@ * For functions not called in fast paths (aka. cold functions) * - PRINTF_FMT * For functions which take printf-style parameters. - * - IDEMPOTENT + * - CONST_FUNCTION * For functions which return the same value for same parameters. * - NEEDED * For functions and variables which must be emitted even if unused. @@ -18,7 +18,7 @@ * For functions and variables which need not be emitted if unused. * - UNUSED * For parameters which are not used. - * - IS_COMPILE_CONSTANT + * - IS_COMPILE_CONSTANT() * For using different tradeoffs for compiletime vs runtime evaluation. * * License: Public Domain diff --git a/lib/ccan/compiler/compiler.h b/lib/ccan/compiler/compiler.h index 74e0f1835c..fcb89c8f2a 100644 --- a/lib/ccan/compiler/compiler.h +++ b/lib/ccan/compiler/compiler.h @@ -61,18 +61,18 @@ #endif #endif -#ifndef IDEMPOTENT +#ifndef CONST_FUNCTION #if HAVE_ATTRIBUTE_CONST /** - * IDEMPOTENT - a function's return depends only on its argument + * CONST_FUNCTION - a function's return depends only on its argument * * This allows the compiler to assume that the function will return the exact * same value for the exact same arguments. This implies that the function * must not use global variables, or dereference pointer arguments. */ -#define IDEMPOTENT __attribute__((const)) +#define CONST_FUNCTION __attribute__((const)) #else -#define IDEMPOTENT +#define CONST_FUNCTION #endif #endif |