diff options
| author | Anoop C S <achiraya@redhat.com> | 2015-05-29 16:33:09 +0530 |
|---|---|---|
| committer | Venky Shankar <vshankar@redhat.com> | 2015-06-01 01:56:28 -0700 |
| commit | eeb356584897ff778ffcbacff1c71d35d277b726 (patch) | |
| tree | 7590bfdcdab0f4105110e1982fd88244ae0648cb /xlators/features/changelog/src/changelog-helpers.c | |
| parent | 754df15ece18864adc5297c292c20a846804cbec (diff) | |
features/changelog: Remove inline keyword to avoid warnings (gcc v5.1.1)
When compiled with gcc5, following warnings were displayed
and volume start failed:
changelog-helpers.h:499:1: warning: inline function 'changelog_dispatch_event'
declared but never defined
changelog_dispatch_event (xlator_t *, changelog_priv_t *, changelog_event_t *);
gf-changelog-journal-handler.c:692:17: warning: 'list_add_tail' is static but
used in inline function 'gf_changelog_queue_journal' which is not static
list_add_tail (&entry->list, &jnl_proc->entries);
Fix is to remove the keyword from function prototype and
definitions.
Change-Id: I188b35b7ca087a94d7a48a052b05a6d845e3b74b
BUG: 1226307
Signed-off-by: Anoop C S <achiraya@redhat.com>
Reviewed-on: http://review.gluster.org/11004
Tested-by: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Venky Shankar <vshankar@redhat.com>
Diffstat (limited to 'xlators/features/changelog/src/changelog-helpers.c')
| -rw-r--r-- | xlators/features/changelog/src/changelog-helpers.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/xlators/features/changelog/src/changelog-helpers.c b/xlators/features/changelog/src/changelog-helpers.c index a604891f0b3..dd09716d800 100644 --- a/xlators/features/changelog/src/changelog-helpers.c +++ b/xlators/features/changelog/src/changelog-helpers.c @@ -100,7 +100,7 @@ changelog_selector_index (unsigned int selector) return (ffs (selector) - 1); } -inline int +int changelog_ev_selected (xlator_t *this, changelog_ev_selector_t *selection, unsigned int selector) @@ -116,7 +116,7 @@ changelog_ev_selected (xlator_t *this, && (selection->ref[idx] > 0)); } -inline void +void changelog_select_event (xlator_t *this, changelog_ev_selector_t *selection, unsigned int selector) @@ -138,7 +138,7 @@ changelog_select_event (xlator_t *this, UNLOCK (&selection->reflock); } -inline void +void changelog_deselect_event (xlator_t *this, changelog_ev_selector_t *selection, unsigned int selector) @@ -160,7 +160,7 @@ changelog_deselect_event (xlator_t *this, UNLOCK (&selection->reflock); } -inline int +int changelog_init_event_selection (xlator_t *this, changelog_ev_selector_t *selection) { @@ -182,7 +182,7 @@ changelog_init_event_selection (xlator_t *this, return 0; } -inline int +int changelog_cleanup_event_selection (xlator_t *this, changelog_ev_selector_t *selection) { @@ -221,7 +221,7 @@ changelog_perform_dispatch (xlator_t *this, rbuf_write_complete (opaque); } -inline void +void changelog_dispatch_event (xlator_t *this, changelog_priv_t *priv, changelog_event_t *ev) { |