diff options
| author | Harshavardhana <harsha@harshavardhana.net> | 2014-05-03 16:17:04 -0700 |
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2014-05-03 19:40:11 -0700 |
| commit | d5918849bebc8df85b017e3eddefaef69f008889 (patch) | |
| tree | 2c13a51909e3dfdb23b02ab4ec341e542f659a9f /xlators/features/changelog/src/changelog.c | |
| parent | aea76f0ec5ca01fd10810d704e5ad00cb4eba2fc (diff) | |
changelog: in C99 `inline` shouldn't be part of function definition
- C99 standard dictates that `inline` should be used only to hint
function implementation not as a definition
- http://clang.llvm.org/compatibility.html#inline
This also leads to run time issues as 'undefined symbols'
- Remember to use adding explicit headers when using an external
function
Change-Id: I66a9030da7ac38ded6e0b6a977182be977382fa2
BUG: 1089172
Signed-off-by: Harshavardhana <harsha@harshavardhana.net>
Reviewed-on: http://review.gluster.org/7655
Reviewed-by: Anand Avati <avati@redhat.com>
Tested-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'xlators/features/changelog/src/changelog.c')
| -rw-r--r-- | xlators/features/changelog/src/changelog.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/xlators/features/changelog/src/changelog.c b/xlators/features/changelog/src/changelog.c index 1253a1a6829..5eb2cd93fd3 100644 --- a/xlators/features/changelog/src/changelog.c +++ b/xlators/features/changelog/src/changelog.c @@ -19,6 +19,7 @@ #include "iobuf.h" #include "changelog-rt.h" +#include "changelog-helpers.h" #include "changelog-encoders.h" #include "changelog-mem-types.h" |