[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[taler-exchange] branch master updated: -code cleanup
|
From: |
gnunet |
|
Subject: |
[taler-exchange] branch master updated: -code cleanup |
|
Date: |
Thu, 17 Nov 2022 17:38:37 +0100 |
This is an automated email from the git hooks/post-receive script.
grothoff pushed a commit to branch master
in repository exchange.
The following commit(s) were added to refs/heads/master by this push:
new 3d4baa99 -code cleanup
3d4baa99 is described below
commit 3d4baa996936b55b595d4f2f619ce47472a21d10
Author: Christian Grothoff <christian@grothoff.org>
AuthorDate: Thu Nov 17 16:58:38 2022 +0100
-code cleanup
---
src/bank-lib/fakebank.c | 7 +-
src/lib/exchange_api_withdraw.c | 2 +-
src/templating/mustach-jansson.c | 156 +++++++++++++++-------------
src/testing/testing_api_cmd_contract_get.c | 1 +
src/testing/testing_api_cmd_exec_closer.c | 1 +
src/testing/testing_api_cmd_purse_deposit.c | 11 +-
src/testing/testing_api_cmd_purse_get.c | 9 +-
src/testing/testing_api_cmd_reserve_get.c | 9 +-
src/testing/testing_api_cmd_wire.c | 6 ++
9 files changed, 104 insertions(+), 98 deletions(-)
diff --git a/src/bank-lib/fakebank.c b/src/bank-lib/fakebank.c
index 34b87be5..b3d3250a 100644
--- a/src/bank-lib/fakebank.c
+++ b/src/bank-lib/fakebank.c
@@ -1484,6 +1484,8 @@ TALER_FAKEBANK_stop (struct TALER_FAKEBANK_Handle *h)
while (NULL != (lp = GNUNET_CONTAINER_heap_remove_root (h->lp_heap)))
lp_trigger (lp,
h);
+ GNUNET_assert (0 ==
+ pthread_mutex_unlock (&h->big_lock));
GNUNET_break (sizeof (val) ==
#ifdef __linux__
write (h->lp_event,
@@ -1492,8 +1494,6 @@ TALER_FAKEBANK_stop (struct TALER_FAKEBANK_Handle *h)
#endif
&val,
sizeof (val)));
- GNUNET_assert (0 ==
- pthread_mutex_unlock (&h->big_lock));
GNUNET_break (0 ==
pthread_join (h->lp_thread,
&ret));
@@ -3012,7 +3012,7 @@ handle_bank_integration (struct TALER_FAKEBANK_Handle *h,
char dummy;
if (1 != sscanf (lp_s,
- "%lld%c",
+ "%llu%c",
&d,
&dummy))
{
@@ -3427,6 +3427,7 @@ post_testing_register (struct TALER_FAKEBANK_Handle *h,
acc = lookup_account (h,
username,
username);
+ GNUNET_assert (NULL != acc);
acc->password = GNUNET_strdup (password);
acc->balance = h->signup_bonus; /* magic money creation! */
}
diff --git a/src/lib/exchange_api_withdraw.c b/src/lib/exchange_api_withdraw.c
index 6643cac6..62c04203 100644
--- a/src/lib/exchange_api_withdraw.c
+++ b/src/lib/exchange_api_withdraw.c
@@ -232,7 +232,7 @@ withdraw_cs_stage_two_callback (
&wh->pd))
{
GNUNET_break (0);
- GNUNET_free (wh);
+ break;
}
wh->wh2 = TALER_EXCHANGE_withdraw2 (wh->exchange,
&wh->pd,
diff --git a/src/templating/mustach-jansson.c b/src/templating/mustach-jansson.c
index 2aed5829..c65fe2b0 100644
--- a/src/templating/mustach-jansson.c
+++ b/src/templating/mustach-jansson.c
@@ -82,7 +82,7 @@ struct JanssonClosure
* Last bang we found.
*/
enum Bang found_bang;
-
+
/**
* Language for i18n lookups.
*/
@@ -129,11 +129,11 @@ find (struct JanssonClosure *e, const char *name)
if (0 == strcmp (bang, "i18n"))
e->found_bang = BANG_I18N;
- else if (0 == strcmp(bang, "stringify"))
+ else if (0 == strcmp (bang, "stringify"))
e->found_bang = BANG_STRINGIFY;
- else if (0 == strcmp(bang, "amount_decimal"))
+ else if (0 == strcmp (bang, "amount_decimal"))
e->found_bang = BANG_AMOUNT_CURRENCY;
- else if (0 == strcmp(bang, "amount_currency"))
+ else if (0 == strcmp (bang, "amount_currency"))
e->found_bang = BANG_AMOUNT_DECIMAL;
}
@@ -157,7 +157,7 @@ find (struct JanssonClosure *e, const char *name)
static int
-start(void *closure)
+start (void *closure)
{
struct JanssonClosure *e = closure;
e->depth = 0;
@@ -174,25 +174,25 @@ static int
emituw (void *closure, const char *buffer, size_t size, int escape, FILE *file)
{
struct JanssonClosure *e = closure;
- if (!escape)
+ if (! escape)
e->writecb (file, buffer, size);
else
do
{
switch (*buffer)
{
- case '<':
- e->writecb (file, "<", 4);
- break;
- case '>':
- e->writecb (file, ">", 4);
- break;
- case '&':
- e->writecb (file, "&", 5);
- break;
- default:
- e->writecb (file, buffer, 1);
- break;
+ case '<':
+ e->writecb (file, "<", 4);
+ break;
+ case '>':
+ e->writecb (file, ">", 4);
+ break;
+ case '&':
+ e->writecb (file, "&", 5);
+ break;
+ default:
+ e->writecb (file, buffer, 1);
+ break;
}
buffer++;
}
@@ -202,10 +202,10 @@ emituw (void *closure, const char *buffer, size_t size,
int escape, FILE *file)
static int
-enter(void *closure, const char *name)
+enter (void *closure, const char *name)
{
struct JanssonClosure *e = closure;
- json_t *o = find(e, name);
+ json_t *o = find (e, name);
if (++e->depth >= MUSTACH_MAX_DEPTH)
return MUSTACH_ERROR_TOO_DEEP;
@@ -277,6 +277,7 @@ next (void *closure)
return 1;
}
+
static int
leave (void *closure)
{
@@ -287,12 +288,14 @@ leave (void *closure)
return 0;
}
+
static void
freecb (void *v)
{
free (v);
}
+
static int
get (void *closure, const char *name, struct mustach_sbuf *sbuf)
{
@@ -300,7 +303,7 @@ get (void *closure, const char *name, struct mustach_sbuf
*sbuf)
json_t *obj;
if ( (0 == strcmp (name, "*") ) &&
- (e->stack[e->depth].is_objiter ) )
+ (e->stack[e->depth].is_objiter) )
{
sbuf->value = json_object_iter_key (e->stack[e->depth].iter);
return MUSTACH_OK;
@@ -310,64 +313,65 @@ get (void *closure, const char *name, struct mustach_sbuf
*sbuf)
{
switch (e->found_bang)
{
- case BANG_I18N:
- case BANG_NONE:
- {
- const char *s = json_string_value (obj);
- if (NULL != s)
- {
- sbuf->value = s;
- return MUSTACH_OK;
- }
- }
- break;
- case BANG_STRINGIFY:
- sbuf->value = json_dumps (obj, JSON_INDENT (2));
- sbuf->freecb = freecb;
- return MUSTACH_OK;
- case BANG_AMOUNT_DECIMAL:
+ case BANG_I18N:
+ case BANG_NONE:
+ {
+ const char *s = json_string_value (obj);
+ if (NULL != s)
{
- char *s;
- char *c;
- if (!json_is_string (obj))
- break;
- s = strdup (json_string_value (obj));
- c = strchr (s, ':');
- if (NULL != c)
- *c = 0;
sbuf->value = s;
- sbuf->freecb = freecb;
- return MUSTACH_OK;
- }
- break;
- case BANG_AMOUNT_CURRENCY:
- {
- const char *s;
- if (!json_is_string (obj))
- break;
- s = json_string_value (obj);
- s = strchr (s, ':');
- if (NULL == s)
- break;
- sbuf->value = s + 1;
return MUSTACH_OK;
}
- break;
- default:
- break;
+ }
+ break;
+ case BANG_STRINGIFY:
+ sbuf->value = json_dumps (obj, JSON_INDENT (2));
+ sbuf->freecb = freecb;
+ return MUSTACH_OK;
+ case BANG_AMOUNT_DECIMAL:
+ {
+ char *s;
+ char *c;
+ if (! json_is_string (obj))
+ break;
+ s = GNUNET_strdup (json_string_value (obj));
+ c = strchr (s, ':');
+ if (NULL != c)
+ *c = 0;
+ sbuf->value = s;
+ sbuf->freecb = freecb;
+ return MUSTACH_OK;
+ }
+ break;
+ case BANG_AMOUNT_CURRENCY:
+ {
+ const char *s;
+ if (! json_is_string (obj))
+ break;
+ s = json_string_value (obj);
+ s = strchr (s, ':');
+ if (NULL == s)
+ break;
+ sbuf->value = s + 1;
+ return MUSTACH_OK;
+ }
+ break;
+ default:
+ break;
}
}
sbuf->value = "";
return MUSTACH_OK;
}
+
static struct mustach_itf itf = {
.start = start,
.put = NULL,
.enter = enter,
.next = next,
.leave = leave,
- .partial =NULL,
+ .partial = NULL,
.get = get,
.emit = NULL,
.stop = NULL
@@ -385,33 +389,41 @@ static struct mustach_itf itfuw = {
.stop = NULL
};
-int fmustach_jansson (const char *template, json_t *root, FILE *file)
+int
+fmustach_jansson (const char *template, json_t *root, FILE *file)
{
struct JanssonClosure e = { 0 };
e.root = root;
- return fmustach(template, &itf, &e, file);
+ return fmustach (template, &itf, &e, file);
}
-int fdmustach_jansson (const char *template, json_t *root, int fd)
+
+int
+fdmustach_jansson (const char *template, json_t *root, int fd)
{
struct JanssonClosure e = { 0 };
e.root = root;
- return fdmustach(template, &itf, &e, fd);
+ return fdmustach (template, &itf, &e, fd);
}
-int mustach_jansson (const char *template, json_t *root, char **result, size_t
*size)
+
+int
+mustach_jansson (const char *template, json_t *root, char **result,
+ size_t *size)
{
struct JanssonClosure e = { 0 };
e.root = root;
e.writecb = NULL;
- return mustach(template, &itf, &e, result, size);
+ return mustach (template, &itf, &e, result, size);
}
-int umustach_jansson (const char *template, json_t *root,
mustach_jansson_write_cb writecb, void *closure)
+
+int
+umustach_jansson (const char *template, json_t *root, mustach_jansson_write_cb
+ writecb, void *closure)
{
struct JanssonClosure e = { 0 };
e.root = root;
e.writecb = writecb;
- return fmustach(template, &itfuw, &e, closure);
+ return fmustach (template, &itfuw, &e, closure);
}
-
diff --git a/src/testing/testing_api_cmd_contract_get.c
b/src/testing/testing_api_cmd_contract_get.c
index 10a43aa3..428bf7e8 100644
--- a/src/testing/testing_api_cmd_contract_get.c
+++ b/src/testing/testing_api_cmd_contract_get.c
@@ -115,6 +115,7 @@ get_cb (void *cls,
}
ref = TALER_TESTING_interpreter_lookup_command (ds->is,
ds->contract_ref);
+ GNUNET_assert (NULL != ref);
if (MHD_HTTP_OK == dr->hr.http_status)
{
const struct TALER_PurseMergePrivateKeyP *mp;
diff --git a/src/testing/testing_api_cmd_exec_closer.c
b/src/testing/testing_api_cmd_exec_closer.c
index 9ee436af..2501b39a 100644
--- a/src/testing/testing_api_cmd_exec_closer.c
+++ b/src/testing/testing_api_cmd_exec_closer.c
@@ -91,6 +91,7 @@ closer_run (void *cls,
rcmd = TALER_TESTING_interpreter_lookup_command (is,
as->reserve_ref);
+ GNUNET_assert (NULL != rcmd);
if (GNUNET_OK !=
TALER_TESTING_get_trait_reserve_pub (rcmd,
&reserve_pubp))
diff --git a/src/testing/testing_api_cmd_purse_deposit.c
b/src/testing/testing_api_cmd_purse_deposit.c
index b056497e..ff8e6d2e 100644
--- a/src/testing/testing_api_cmd_purse_deposit.c
+++ b/src/testing/testing_api_cmd_purse_deposit.c
@@ -163,7 +163,7 @@ deposit_cb (void *cls,
purse_cmd = TALER_TESTING_interpreter_lookup_command (ds->is,
ds->purse_ref);
-
+ GNUNET_assert (NULL != purse_cmd);
if (GNUNET_OK !=
TALER_TESTING_get_trait_reserve_sig (purse_cmd,
&reserve_sig))
@@ -270,6 +270,7 @@ deposit_run (void *cls,
purse_cmd = TALER_TESTING_interpreter_lookup_command (is,
ds->purse_ref);
+ GNUNET_assert (NULL != purse_cmd);
if (GNUNET_OK !=
TALER_TESTING_get_trait_purse_pub (purse_cmd,
&purse_pub))
@@ -291,13 +292,7 @@ deposit_run (void *cls,
coin_cmd = TALER_TESTING_interpreter_lookup_command (is,
cr->command_ref);
- if (NULL == coin_cmd)
- {
- GNUNET_break (0);
- TALER_TESTING_interpreter_fail (is);
- return;
- }
-
+ GNUNET_assert (NULL != coin_cmd);
if ( (GNUNET_OK !=
TALER_TESTING_get_trait_coin_priv (coin_cmd,
cr->coin_index,
diff --git a/src/testing/testing_api_cmd_purse_get.c
b/src/testing/testing_api_cmd_purse_get.c
index 61873721..3e7da38f 100644
--- a/src/testing/testing_api_cmd_purse_get.c
+++ b/src/testing/testing_api_cmd_purse_get.c
@@ -188,13 +188,7 @@ status_run (void *cls,
create_purse
= TALER_TESTING_interpreter_lookup_command (is,
ss->purse_reference);
-
- if (NULL == create_purse)
- {
- GNUNET_break (0);
- TALER_TESTING_interpreter_fail (is);
- return;
- }
+ GNUNET_assert (NULL != create_purse);
if (GNUNET_OK !=
TALER_TESTING_get_trait_purse_pub (create_purse,
&ss->purse_pub))
@@ -311,6 +305,7 @@ finish_run (void *cls,
poll_purse
= TALER_TESTING_interpreter_lookup_command (is,
ps->poll_reference);
+ GNUNET_assert (NULL != poll_purse);
GNUNET_assert (poll_purse->run == &status_run);
ss = poll_purse->cls;
if (NULL == ss->pgh)
diff --git a/src/testing/testing_api_cmd_reserve_get.c
b/src/testing/testing_api_cmd_reserve_get.c
index f9b8ff6c..22c29a3b 100644
--- a/src/testing/testing_api_cmd_reserve_get.c
+++ b/src/testing/testing_api_cmd_reserve_get.c
@@ -183,13 +183,7 @@ status_run (void *cls,
create_reserve
= TALER_TESTING_interpreter_lookup_command (is,
ss->reserve_reference);
-
- if (NULL == create_reserve)
- {
- GNUNET_break (0);
- TALER_TESTING_interpreter_fail (is);
- return;
- }
+ GNUNET_assert (NULL != create_reserve);
if (GNUNET_OK !=
TALER_TESTING_get_trait_reserve_pub (create_reserve,
&ss->reserve_pubp))
@@ -328,6 +322,7 @@ finish_run (void *cls,
poll_reserve
= TALER_TESTING_interpreter_lookup_command (is,
ps->poll_reference);
+ GNUNET_assert (NULL != poll_reserve);
GNUNET_assert (poll_reserve->run == &status_run);
ss = poll_reserve->cls;
if (NULL == ss->rsh)
diff --git a/src/testing/testing_api_cmd_wire.c
b/src/testing/testing_api_cmd_wire.c
index 4b0a177b..6e44403b 100644
--- a/src/testing/testing_api_cmd_wire.c
+++ b/src/testing/testing_api_cmd_wire.c
@@ -105,6 +105,12 @@ wire_cb (void *cls,
char *method;
method = TALER_payto_get_method (accounts[i].payto_uri);
+ if (NULL == method)
+ {
+ GNUNET_break (0);
+ TALER_TESTING_interpreter_fail (ws->is);
+ return;
+ }
if (0 == strcmp (ws->expected_method,
method))
{
--
To stop receiving notification emails like this one, please contact
gnunet@gnunet.org.
| [Prev in Thread] |
Current Thread |
[Next in Thread] |
- [taler-exchange] branch master updated: -code cleanup,
gnunet <=