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 06:56:04 -0700 |
commit | f266dc6be5b401a7a813adc958d978eb9207cd63 (patch) | |
tree | 8a13a05e788c22d2223dd66c05319f53c7095efa | |
parent | cabb28a028b70172c45c72598e1ac43fbed08411 (diff) |
features/changelog: Remove inline keyword to avoid warnings (gcc v5.1.1)v3.7.1
Backport of http://review.gluster.org/11004
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: 1226853
Signed-off-by: Anoop C S <achiraya@redhat.com>
Reviewed-on: http://review.gluster.org/11030
Tested-by: NetBSD Build System <jenkins@build.gluster.org>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>
5 files changed, 16 insertions, 16 deletions
diff --git a/xlators/features/changelog/lib/src/gf-changelog-journal-handler.c b/xlators/features/changelog/lib/src/gf-changelog-journal-handler.c index e5bc38f865d..2975b06757f 100644 --- a/xlators/features/changelog/lib/src/gf-changelog-journal-handler.c +++ b/xlators/features/changelog/lib/src/gf-changelog-journal-handler.c @@ -671,7 +671,7 @@ gf_changelog_process (void *data) return NULL; } -inline void +void gf_changelog_queue_journal (gf_changelog_processor_t *jnl_proc, changelog_event_t *event) { diff --git a/xlators/features/changelog/lib/src/gf-changelog-reborp.c b/xlators/features/changelog/lib/src/gf-changelog-reborp.c index 457d829e69b..2a1e56ae79e 100644 --- a/xlators/features/changelog/lib/src/gf-changelog-reborp.c +++ b/xlators/features/changelog/lib/src/gf-changelog-reborp.c @@ -197,7 +197,7 @@ gf_changelog_reborp_init_rpc_listner (xlator_t *this, * * @FIXME: cleanup this bugger once server filters events. */ -inline void +void gf_changelog_invoke_callback (gf_changelog_t *entry, struct iovec **vec, int payloadcnt) { @@ -236,7 +236,7 @@ __is_expected_sequence (struct gf_event_list *ev, struct gf_event *event) return (ev->next_seq == event->seq); } -inline int +int __can_process_event (struct gf_event_list *ev, struct gf_event **event) { *event = list_first_entry (&ev->events, struct gf_event, list); diff --git a/xlators/features/changelog/src/changelog-helpers.c b/xlators/features/changelog/src/changelog-helpers.c index 847b9d2a32a..7e518f1a0c7 100644 --- a/xlators/features/changelog/src/changelog-helpers.c +++ b/xlators/features/changelog/src/changelog-helpers.c @@ -105,7 +105,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) @@ -121,7 +121,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) @@ -143,7 +143,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) @@ -165,7 +165,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) { @@ -187,7 +187,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) { @@ -226,7 +226,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) { diff --git a/xlators/features/changelog/src/changelog-helpers.h b/xlators/features/changelog/src/changelog-helpers.h index 4416b9cdfdb..6219d8d9a4b 100644 --- a/xlators/features/changelog/src/changelog-helpers.h +++ b/xlators/features/changelog/src/changelog-helpers.h @@ -486,17 +486,17 @@ changelog_fill_entry_buf (call_frame_t *frame, xlator_t *this, loc_t *loc, changelog_local_t **local); /* event selection routines */ -inline void changelog_select_event (xlator_t *, +void changelog_select_event (xlator_t *, changelog_ev_selector_t *, unsigned int); -inline void changelog_deselect_event (xlator_t *, +void changelog_deselect_event (xlator_t *, changelog_ev_selector_t *, unsigned int); -inline int changelog_init_event_selection (xlator_t *, +int changelog_init_event_selection (xlator_t *, changelog_ev_selector_t *); -inline int changelog_cleanup_event_selection (xlator_t *, +int changelog_cleanup_event_selection (xlator_t *, changelog_ev_selector_t *); -inline int changelog_ev_selected (xlator_t *, +int changelog_ev_selected (xlator_t *, changelog_ev_selector_t *, unsigned int); -inline void +void changelog_dispatch_event (xlator_t *, changelog_priv_t *, changelog_event_t *); changelog_inode_ctx_t * diff --git a/xlators/features/changelog/src/changelog-rpc.c b/xlators/features/changelog/src/changelog-rpc.c index ffbc61e40f9..c7c0fffa9f5 100644 --- a/xlators/features/changelog/src/changelog-rpc.c +++ b/xlators/features/changelog/src/changelog-rpc.c @@ -187,7 +187,7 @@ changelog_rpc_clnt_cleanup (changelog_rpc_clnt_t *crpc) GF_FREE (crpc); } -inline changelog_rpc_clnt_t * +static inline changelog_rpc_clnt_t * changelog_rpc_clnt_init (xlator_t *this, changelog_probe_req *rpc_req, changelog_clnt_t *c_clnt) { |