diff options
author | Christian Grothoff <christian@grothoff.org> | 2019-06-03 00:25:11 +0200 |
---|---|---|
committer | Christian Grothoff <christian@grothoff.org> | 2019-06-03 00:25:11 +0200 |
commit | f7255931558bf788073af15b7e433680a10f218c (patch) | |
tree | 6e68d095eb0b2ca9a0660f594cac75ded0231d54 | |
parent | f2ba3508b31b2bfc5a3eb2db56600107fde6849e (diff) | |
download | exchange-f7255931558bf788073af15b7e433680a10f218c.tar.xz |
eliminate useless global variable TFH_handle_history_range_step
-rw-r--r-- | src/bank-lib/fakebank.c | 3 | ||||
-rw-r--r-- | src/bank-lib/fakebank.h | 7 | ||||
-rw-r--r-- | src/bank-lib/fakebank_history.c | 16 |
3 files changed, 5 insertions, 21 deletions
diff --git a/src/bank-lib/fakebank.c b/src/bank-lib/fakebank.c index 71dd8bace..b6aee7f41 100644 --- a/src/bank-lib/fakebank.c +++ b/src/bank-lib/fakebank.c @@ -755,7 +755,6 @@ handle_history_range (struct TALER_FAKEBANK_Handle *h, hrd.end.abs_value_us = end_stamp * 1000LL * 1000LL; ha.range = &hrd; - pos = NULL; /* hunt for 'pos' in the Transaction(s) LL. */ for (pos = h->transactions_head; NULL != pos; @@ -768,7 +767,7 @@ handle_history_range (struct TALER_FAKEBANK_Handle *h, pos, &ha, &TFH_handle_history_range_skip, - TFH_handle_history_range_step, + &TFH_handle_history_range_skip, &TFH_handle_history_range_advance); } diff --git a/src/bank-lib/fakebank.h b/src/bank-lib/fakebank.h index f547db52d..ff8ea7165 100644 --- a/src/bank-lib/fakebank.h +++ b/src/bank-lib/fakebank.h @@ -312,11 +312,4 @@ struct Transaction * TFH_handle_history_range_skip (const struct HistoryArgs *ha, const struct Transaction *pos); -/** - * Iterates on the "next" element to be processed. To - * be used when the current element _gets_ inserted in the result. - * Same implementation of the "skip" counterpart, as /history-range - * does not have the notion of count/delta. - */ -Step TFH_handle_history_range_step; #endif diff --git a/src/bank-lib/fakebank_history.c b/src/bank-lib/fakebank_history.c index 17960a4e5..a3553b65b 100644 --- a/src/bank-lib/fakebank_history.c +++ b/src/bank-lib/fakebank_history.c @@ -135,18 +135,11 @@ TFH_handle_history_range_skip (const struct HistoryArgs *ha, return pos->next; } -/** - * Iterates on the "next" element to be processed. To - * be used when the current element _gets_ inserted in the result. - * Same implementation of the "skip" counterpart, as /history-range - * does not have the notion of count/delta. - */ -Step TFH_handle_history_range_step = &TFH_handle_history_range_skip; /** * Actual history response builder. * - * @param pos first (included) element in the result set. + * @param pos first (included) element in the result set, NULL if history is empty * @param ha history arguments. * @param caller_name which function is building the history. * @return MHD_YES / MHD_NO, after having enqueued the response @@ -168,8 +161,9 @@ TFH_build_history_response (struct MHD_Connection *connection, json_t *jresponse; int ret; - while (advance (ha, - pos)) + while ( (NULL != pos) && + advance (ha, + pos) ) { json_t *trans; char *subject; @@ -423,5 +417,3 @@ TFH_parse_history_common_args (struct MHD_Connection *connection, return GNUNET_OK; } - - |