From 0773ca67fdb60a142207759fa6c07a69882ce59c Mon Sep 17 00:00:00 2001 From: Jeff Darcy Date: Tue, 28 Jul 2015 12:11:12 -0400 Subject: all: reduce "inline" usage There are three kinds of inline functions: plain inline, extern inline, and static inline. All three have been removed from .c files, except those in "contrib" which aren't our problem. Inlines in .h files, which are overwhelmingly "static inline" already, have generally been left alone. Over time we should be able to "lower" these into .c files, but that has to be done in a case-by-case fashion requiring more manual effort. This part was easy to do automatically without (as far as I can tell) any ill effect. In the process, several pieces of dead code were flagged by the compiler, and were removed. Change-Id: I56a5e614735c9e0a6ee420dab949eac22e25c155 BUG: 1245331 Signed-off-by: Jeff Darcy Reviewed-on: http://review.gluster.org/11769 Tested-by: NetBSD Build System Reviewed-by: Dan Lambright Reviewed-by: Kaleb KEITHLEY Reviewed-by: Niels de Vos Reviewed-by: Raghavendra Bhat Reviewed-by: Venky Shankar --- .../changelog/lib/src/gf-changelog-reborp.c | 53 +--------------------- 1 file changed, 1 insertion(+), 52 deletions(-) (limited to 'xlators/features/changelog/lib') diff --git a/xlators/features/changelog/lib/src/gf-changelog-reborp.c b/xlators/features/changelog/lib/src/gf-changelog-reborp.c index e6bb19e3696..665a6d0a5da 100644 --- a/xlators/features/changelog/lib/src/gf-changelog-reborp.c +++ b/xlators/features/changelog/lib/src/gf-changelog-reborp.c @@ -93,55 +93,6 @@ gf_changelog_connection_janitor (void *arg) return NULL; } -static inline void -__gf_changelog_set_conn_state (gf_changelog_t *entry, - gf_changelog_conn_state_t state) -{ - entry->connstate = state; -} - -/** - * state check login to gaurd access object after free - */ -static inline void -gf_changelog_check_event (gf_private_t *priv, - gf_changelog_t *entry, rpcsvc_event_t event) -{ - gf_boolean_t needfree = _gf_false; - gf_changelog_conn_state_t laststate; - /** - * need to handle couple of connection states to gaurd correct - * freeing of object. - */ - LOCK (&entry->statelock); - { - laststate = entry->connstate; - if (event == RPCSVC_EVENT_ACCEPT) { - __gf_changelog_set_conn_state - (entry, GF_CHANGELOG_CONN_STATE_ACCEPTED); - - if (laststate == GF_CHANGELOG_CONN_STATE_DISCONNECTED) - needfree = _gf_true; - } - - if (event == RPCSVC_EVENT_DISCONNECT) { - __gf_changelog_set_conn_state - (entry, GF_CHANGELOG_CONN_STATE_DISCONNECTED); - - if (laststate == GF_CHANGELOG_CONN_STATE_ACCEPTED) - needfree = _gf_true; - } - } - UNLOCK (&entry->statelock); - - /** - * TODO: - * Handle the race between ACCEPT and DISCONNECT in the - * reconnect code. So purging of entry is deliberately - * avoided here. It will be handled in the reconnect code. - */ -} - int gf_changelog_reborp_rpcsvc_notify (rpcsvc_t *rpc, void *mydata, rpcsvc_event_t event, void *data) @@ -181,8 +132,6 @@ gf_changelog_reborp_rpcsvc_notify (rpcsvc_t *rpc, void *mydata, break; } - /* gf_changelog_check_event (priv, entry, event); */ - return 0; } @@ -238,7 +187,7 @@ gf_changelog_invoke_callback (gf_changelog_t *entry, * dynamically allocated and ordered. */ -inline int +int __is_expected_sequence (struct gf_event_list *ev, struct gf_event *event) { return (ev->next_seq == event->seq); -- cgit