summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--src/include/taler_bank_service.h4
-rw-r--r--src/include/taler_exchange_service.h8
-rw-r--r--src/include/taler_testing_lib.h13
-rw-r--r--src/lib/auditor_api_handle.c2
-rw-r--r--src/lib/exchange_api_handle.c20
-rw-r--r--src/lib/exchange_api_handle.h2
-rw-r--r--src/lib/testing_api_cmd_auditor_exchanges.c4
-rw-r--r--src/lib/testing_api_cmd_deposit.c2
-rw-r--r--src/lib/testing_api_cmd_payback.c4
-rw-r--r--src/lib/testing_api_cmd_refresh.c4
-rw-r--r--src/lib/testing_api_cmd_status.c1
-rw-r--r--src/lib/testing_api_cmd_track.c9
-rw-r--r--src/lib/testing_api_cmd_withdraw.c5
-rw-r--r--src/lib/testing_api_helpers_auditor.c9
-rw-r--r--src/lib/testing_api_helpers_exchange.c11
-rw-r--r--src/lib/testing_api_trait_cmd.c9
16 files changed, 52 insertions, 55 deletions
diff --git a/src/include/taler_bank_service.h b/src/include/taler_bank_service.h
index 510712b9c..dda338404 100644
--- a/src/include/taler_bank_service.h
+++ b/src/include/taler_bank_service.h
@@ -236,11 +236,11 @@ TALER_BANK_execute_wire_transfer (struct GNUNET_CURL_Context *ctx,
* or ultimately fails. Until this has been done, the transaction is
* in limbo (i.e. may or may not have been committed).
*
- * @param eh execution to cancel
+ * @param weh execution to cancel
*/
void
TALER_BANK_execute_wire_transfer_cancel (struct
- TALER_BANK_WireExecuteHandle *eh);
+ TALER_BANK_WireExecuteHandle *weh);
/* ********************* /taler/credits *********************** */
diff --git a/src/include/taler_exchange_service.h b/src/include/taler_exchange_service.h
index de1daf1ef..253647428 100644
--- a/src/include/taler_exchange_service.h
+++ b/src/include/taler_exchange_service.h
@@ -1110,7 +1110,7 @@ struct TALER_EXCHANGE_ReserveHistory
* @param http_status HTTP response code, #MHD_HTTP_OK (200) for successful status request
* 0 if the exchange's reply is bogus (fails to follow the protocol)
* @param ec taler-specific error code, #TALER_EC_NONE on success
- * @param[in] json original response in JSON format (useful only for diagnostics)
+ * @param json original response in JSON format (useful only for diagnostics)
* @param balance current balance in the reserve, NULL on error
* @param history_length number of entries in the transaction history, 0 on error
* @param history detailed transaction history, NULL on error
@@ -1648,7 +1648,7 @@ struct TALER_EXCHANGE_TrackTransactionHandle;
* @param wtid wire transfer identifier used by the exchange, NULL if exchange did not
* yet execute the transaction
* @param execution_time actual or planned execution time for the wire transfer
- * @param coin_contribution contribution to the @a total_amount of the deposited coin (may be NULL)
+ * @param coin_contribution contribution to the total amount by this coin (can be NULL)
*/
typedef void
(*TALER_EXCHANGE_TrackTransactionCallback)(void *cls,
@@ -1667,7 +1667,9 @@ typedef void
/**
- * Obtain the wire transfer details for a given transaction.
+ * Obtain the wire transfer details for a given transaction. Tells the client
+ * which aggregate wire transfer the deposit operation identified by @a coin_pub,
+ * @a merchant_priv and @a h_contract_terms contributed to.
*
* @param exchange the exchange to query
* @param merchant_priv the merchant's private key
diff --git a/src/include/taler_testing_lib.h b/src/include/taler_testing_lib.h
index c1c2becc3..14e09c911 100644
--- a/src/include/taler_testing_lib.h
+++ b/src/include/taler_testing_lib.h
@@ -499,12 +499,12 @@ struct TALER_TESTING_Command
/**
* Lookup command by label.
*
- * @param i interpreter state.
+ * @param is interpreter state.
* @param label label of the command to lookup.
* @return the command, if it is found, or NULL.
*/
const struct TALER_TESTING_Command *
-TALER_TESTING_interpreter_lookup_command (struct TALER_TESTING_Interpreter *i,
+TALER_TESTING_interpreter_lookup_command (struct TALER_TESTING_Interpreter *is,
const char *label);
/**
@@ -717,7 +717,7 @@ TALER_TESTING_setup_with_exchange_cfg (void *cls,
*
* @param main_cb main method.
* @param main_cb_cls main method closure.
- * @param config_filename configuration file name. Is is used
+ * @param config_file configuration file name. Is is used
* by both this function and the exchange itself. In the
* first case it gives out the exchange port number and
* the exchange base URL so as to check whether the port
@@ -754,7 +754,7 @@ TALER_TESTING_setup_with_auditor_and_exchange_cfg (void *cls,
*
* @param main_cb main method.
* @param main_cb_cls main method closure.
- * @param config_filename configuration file name. Is is used
+ * @param config_file configuration file name. Is is used
* by both this function and the exchange itself. In the
* first case it gives out the exchange port number and
* the exchange base URL so as to check whether the port
@@ -1180,10 +1180,10 @@ TALER_TESTING_cmd_exec_auditor_sign (const char *label,
* the desired amount as string.
*
* @param label command label.
+ * @param reserve_reference command providing us with a reserve to withdraw from
* @param amount how much we withdraw.
* @param expected_response_code which HTTP response code
* we expect from the exchange.
- *
* @return the withdraw command to be executed by the interpreter.
*/
struct TALER_TESTING_Command
@@ -1884,7 +1884,6 @@ TALER_TESTING_make_trait_bank_row (const uint64_t *row);
*
* @param index reserve priv's index number.
* @param reserve_priv reserve private key to offer.
- *
* @return the trait.
*/
struct TALER_TESTING_Trait
@@ -1912,7 +1911,7 @@ TALER_TESTING_get_trait_reserve_priv (const struct TALER_TESTING_Command *cmd,
* Offer a reserve public key.
*
* @param index reserve pubs's index number.
- * @param reserve_priv reserve public key to offer.
+ * @param reserve_pub reserve public key to offer.
* @return the trait.
*/
struct TALER_TESTING_Trait
diff --git a/src/lib/auditor_api_handle.c b/src/lib/auditor_api_handle.c
index aa0489a30..d38de34ff 100644
--- a/src/lib/auditor_api_handle.c
+++ b/src/lib/auditor_api_handle.c
@@ -274,7 +274,7 @@ request_version (void *cls);
*
* @param cls the `struct VersionRequest`
* @param response_code HTTP response code, 0 on error
- * @param resp_obj parsed JSON result, NULL on error
+ * @param gresp_obj parsed JSON result, NULL on error, must be a `const json_t *`
*/
static void
version_completed_cb (void *cls,
diff --git a/src/lib/exchange_api_handle.c b/src/lib/exchange_api_handle.c
index 41aed278c..9ee64c4c5 100644
--- a/src/lib/exchange_api_handle.c
+++ b/src/lib/exchange_api_handle.c
@@ -296,7 +296,7 @@ TEAH_acc_confirmation_cb (void *cls,
/**
* Iterate over all available auditors for @a h, calling
- * @param ah and giving it a chance to start a deposit
+ * @a ac and giving it a chance to start a deposit
* confirmation interaction.
*
* @param h exchange to go over auditors for
@@ -529,7 +529,7 @@ EXITIF_exit:
* Parse a exchange's auditor information encoded in JSON.
*
* @param[out] auditor where to return the result
- * @param check_sig should we check signatures
+ * @param check_sigs should we check signatures
* @param[in] auditor_obj json to parse
* @param key_data information about denomination keys
* @return #GNUNET_OK if all is fine, #GNUNET_SYSERR if the signature is
@@ -758,14 +758,14 @@ update_auditors (struct TALER_EXCHANGE_Handle *exchange)
/**
* Compare two denomination keys.
*
- * @param denoma first denomination key
- * @param denomb second denomination key
+ * @param denom1 first denomination key
+ * @param denom2 second denomination key
* @return 0 if the two keys are equal (not necessarily
* the same object), 1 otherwise.
*/
static unsigned int
-TALER_denoms_cmp (struct TALER_EXCHANGE_DenomPublicKey *denom1,
- struct TALER_EXCHANGE_DenomPublicKey *denom2)
+denoms_cmp (struct TALER_EXCHANGE_DenomPublicKey *denom1,
+ struct TALER_EXCHANGE_DenomPublicKey *denom2)
{
struct GNUNET_CRYPTO_RsaPublicKey *tmp1;
struct GNUNET_CRYPTO_RsaPublicKey *tmp2;
@@ -806,7 +806,7 @@ TALER_denoms_cmp (struct TALER_EXCHANGE_DenomPublicKey *denom1,
* @param[in] resp_obj JSON object to parse
* @param check_sig #GNUNET_YES if we should check the signature
* @param[out] key_data where to store the results we decoded
- * @param[out] where to store version compatibility data
+ * @param[out] vc where to store version compatibility data
* @return #GNUNET_OK on success, #GNUNET_SYSERR on error
* (malformed JSON)
*/
@@ -948,8 +948,8 @@ decode_keys_json (const json_t *resp_obj,
j<key_data->num_denom_keys;
j++)
{
- if (0 == TALER_denoms_cmp (&dk,
- &key_data->denom_keys[j]))
+ if (0 == denoms_cmp (&dk,
+ &key_data->denom_keys[j]))
{
found = GNUNET_YES;
break;
@@ -1521,7 +1521,7 @@ header_cb (char *buffer,
* tolerated (i.e. by re-downloading instead).
*
* @param exchange which exchange's key and wire data should be deserialized
- * @return data the data to deserialize
+ * @param data the data to deserialize
*/
static void
deserialize_data (struct TALER_EXCHANGE_Handle *exchange,
diff --git a/src/lib/exchange_api_handle.h b/src/lib/exchange_api_handle.h
index 1aefbd655..e0acc4e6d 100644
--- a/src/lib/exchange_api_handle.h
+++ b/src/lib/exchange_api_handle.h
@@ -92,7 +92,7 @@ TEAH_acc_confirmation_cb (void *cls,
/**
* Iterate over all available auditors for @a h, calling
- * @param ah and giving it a chance to start a deposit
+ * @a ac and giving it a chance to start a deposit
* confirmation interaction.
*
* @param h exchange to go over auditors for
diff --git a/src/lib/testing_api_cmd_auditor_exchanges.c b/src/lib/testing_api_cmd_auditor_exchanges.c
index b51a9276b..d021b0d34 100644
--- a/src/lib/testing_api_cmd_auditor_exchanges.c
+++ b/src/lib/testing_api_cmd_auditor_exchanges.c
@@ -115,7 +115,9 @@ do_retry (void *cls)
* @param cls closure.
* @param http_status HTTP response code.
* @param ec taler-specific error code.
- * @param obj raw response from the auditor.
+ * @param num_exchanges length of the @a ei array
+ * @param ei array with information about the exchanges
+ * @param raw_response raw response from the auditor.
*/
static void
exchanges_cb (void *cls,
diff --git a/src/lib/testing_api_cmd_deposit.c b/src/lib/testing_api_cmd_deposit.c
index 94636b7cf..c85cb58cf 100644
--- a/src/lib/testing_api_cmd_deposit.c
+++ b/src/lib/testing_api_cmd_deposit.c
@@ -363,7 +363,7 @@ deposit_run (void *cls,
* Free the state of a "deposit" CMD, and possibly cancel a
* pending operation thereof.
*
- * @param cls closure, typically a #struct WireState.
+ * @param cls closure, must be a `struct DepositState`.
* @param cmd the command which is being cleaned up.
*/
static void
diff --git a/src/lib/testing_api_cmd_payback.c b/src/lib/testing_api_cmd_payback.c
index 1fdd2bfea..41e60b04d 100644
--- a/src/lib/testing_api_cmd_payback.c
+++ b/src/lib/testing_api_cmd_payback.c
@@ -408,14 +408,13 @@ payback_run (void *cls,
/**
* Cleanup the state.
*
- * @param cls closure, typically a #struct WireState.
+ * @param cls closure, must be a `struct RevokeState`.
* @param cmd the command which is being cleaned up.
*/
static void
revoke_cleanup (void *cls,
const struct TALER_TESTING_Command *cmd)
{
-
struct RevokeState *rs = cls;
if (NULL != rs->revoke_proc)
@@ -426,7 +425,6 @@ revoke_cleanup (void *cls,
GNUNET_OS_process_destroy (rs->revoke_proc);
rs->revoke_proc = NULL;
}
-
GNUNET_free_non_null (rs->dhks);
GNUNET_free (rs);
}
diff --git a/src/lib/testing_api_cmd_refresh.c b/src/lib/testing_api_cmd_refresh.c
index 44911a6df..c4dcc3c62 100644
--- a/src/lib/testing_api_cmd_refresh.c
+++ b/src/lib/testing_api_cmd_refresh.c
@@ -1022,7 +1022,7 @@ refresh_melt_run (void *cls,
* Free the "refresh melt" CMD state, and possibly cancel a
* pending operation thereof.
*
- * @param cls closure, typically a #struct RefreshMeltState.
+ * @param cls closure, must be a `struct RefreshMeltState`.
* @param cmd the command which is being cleaned up.
*/
static void
@@ -1099,7 +1099,7 @@ refresh_melt_traits (void *cls,
/**
* Parse list of amounts for melt operation.
*
- * @param rms[in,out] where to store the list
+ * @param[in,out] rms where to store the list
* @param ap NULL-termianted list of amounts to be melted (one per fresh coin)
* @return #GNUNET_OK on success
*/
diff --git a/src/lib/testing_api_cmd_status.c b/src/lib/testing_api_cmd_status.c
index 757ca9951..41d2c326e 100644
--- a/src/lib/testing_api_cmd_status.c
+++ b/src/lib/testing_api_cmd_status.c
@@ -68,6 +68,7 @@ struct StatusState
* @param cls closure.
* @param http_status HTTP response code.
* @param ec taler-specific error code.
+ * @param json original JSON response from the exchange
* @param balance current balance in the reserve, NULL on error.
* @param history_length number of entries in the transaction
* history, 0 on error.
diff --git a/src/lib/testing_api_cmd_track.c b/src/lib/testing_api_cmd_track.c
index bb70c5bad..51ad45be4 100644
--- a/src/lib/testing_api_cmd_track.c
+++ b/src/lib/testing_api_cmd_track.c
@@ -143,17 +143,15 @@ struct TrackTransferState
* @param cls closure.
* @param http_status HTTP status code we got.
* @param ec taler-specific error code.
+ * @param exchange_pub public key of the exchange
* @param json original json reply (may include signatures, those
* have then been validated already).
* @param wtid wire transfer identifier, NULL if exchange did not
* execute the transaction yet.
* @param execution_time actual or planned execution time for the
* wire transfer.
- * @param coin_contribution contribution to the @a total_amount of
+ * @param coin_contribution contribution to the total amount of
* the deposited coin (can be NULL).
- * @param total_amount total amount of the wire transfer, or NULL
- * if the exchange could not provide any @a wtid (set only
- * if @a http_status is #MHD_HTTP_OK).
*/
static void
deposit_wtid_cb (void *cls,
@@ -169,6 +167,8 @@ deposit_wtid_cb (void *cls,
struct TALER_TESTING_Interpreter *is = tts->is;
struct TALER_TESTING_Command *cmd = &is->commands[is->ip];
+ (void) coin_contribution;
+ (void) exchange_pub;
tts->tth = NULL;
if (tts->expected_response_code != http_status)
{
@@ -503,6 +503,7 @@ track_transfer_cb (void *cls,
struct TALER_TESTING_Command *cmd = &is->commands[is->ip];
struct TALER_Amount expected_amount;
+ (void) exchange_pub;
tts->tth = NULL;
if (tts->expected_response_code != http_status)
{
diff --git a/src/lib/testing_api_cmd_withdraw.c b/src/lib/testing_api_cmd_withdraw.c
index 460a55d4e..b6d8b32ab 100644
--- a/src/lib/testing_api_cmd_withdraw.c
+++ b/src/lib/testing_api_cmd_withdraw.c
@@ -349,7 +349,6 @@ withdraw_cleanup (void *cls,
* @param[out] ret result (could be anything)
* @param trait name of the trait
* @param index index number of the object to offer.
- *
* @return #GNUNET_OK on success
*/
static int
@@ -430,10 +429,10 @@ withdraw_traits (void *cls,
* the desired amount as string.
*
* @param label command label.
+ * @param reserve_reference command providing us with a reserve to withdraw from
* @param amount how much we withdraw.
* @param expected_response_code which HTTP response code
* we expect from the exchange.
- *
* @return the withdraw command to be executed by the interpreter.
*/
struct TALER_TESTING_Command
@@ -446,7 +445,6 @@ TALER_TESTING_cmd_withdraw_amount (const char *label,
ws = GNUNET_new (struct WithdrawState);
ws->reserve_reference = reserve_reference;
-
if (GNUNET_OK !=
TALER_string_to_amount (amount,
&ws->amount))
@@ -457,7 +455,6 @@ TALER_TESTING_cmd_withdraw_amount (const char *label,
label);
GNUNET_assert (0);
}
-
ws->expected_response_code = expected_response_code;
{
struct TALER_TESTING_Command cmd = {
diff --git a/src/lib/testing_api_helpers_auditor.c b/src/lib/testing_api_helpers_auditor.c
index ff85c0c85..26585f5b9 100644
--- a/src/lib/testing_api_helpers_auditor.c
+++ b/src/lib/testing_api_helpers_auditor.c
@@ -105,10 +105,9 @@ struct MainWrapperContext
* @param compat protocol compatibility information
*/
static void
-auditor_version_cb
- (void *cls,
- const struct TALER_AUDITOR_VersionInformation *vi,
- enum TALER_AUDITOR_VersionCompatibility compat)
+auditor_version_cb (void *cls,
+ const struct TALER_AUDITOR_VersionInformation *vi,
+ enum TALER_AUDITOR_VersionCompatibility compat)
{
struct TALER_TESTING_Interpreter *is = cls;
@@ -126,7 +125,7 @@ auditor_version_cb
* Setup the @a is 'auditor' member before running the main test loop.
*
* @param cls must be a `struct MainWrapperContext *`
- * @param is[in,out] interpreter state to setup
+ * @param[in,out] is interpreter state to setup
*/
static void
auditor_main_wrapper (void *cls,
diff --git a/src/lib/testing_api_helpers_exchange.c b/src/lib/testing_api_helpers_exchange.c
index 1aeff16d8..5979fc015 100644
--- a/src/lib/testing_api_helpers_exchange.c
+++ b/src/lib/testing_api_helpers_exchange.c
@@ -376,8 +376,7 @@ struct SignInfo
* Sign the keys for an exchange given configuration @a cfg.
* The information to be signed must be in a file "auditor.in".
*
- * @param cls[in,out] a `struct SignInfo` with
- * further paramters
+ * @param[in,out] cls a `struct SignInfo` with further paramters
* @param cfg configuration to use
* @return #GNUNET_OK on success
*/
@@ -680,17 +679,17 @@ TALER_TESTING_wait_auditor_ready (const char *base_url)
int
TALER_TESTING_setup_with_exchange (TALER_TESTING_Main main_cb,
void *main_cb_cls,
- const char *config_filename)
+ const char *config_file)
{
struct TALER_TESTING_SetupContext setup_ctx = {
- .config_filename = config_filename,
+ .config_filename = config_file,
.main_cb = main_cb,
.main_cb_cls = main_cb_cls
};
int result;
result =
- GNUNET_CONFIGURATION_parse_and_run (config_filename,
+ GNUNET_CONFIGURATION_parse_and_run (config_file,
&TALER_TESTING_setup_with_exchange_cfg,
&setup_ctx);
if (GNUNET_OK != result)
@@ -917,7 +916,7 @@ TALER_TESTING_setup_with_auditor_and_exchange_cfg (void *cls,
*
* @param main_cb main method.
* @param main_cb_cls main method closure.
- * @param config_filename configuration file name. Is is used
+ * @param config_file configuration file name. Is is used
* by both this function and the exchange itself. In the
* first case it gives out the exchange port number and
* the exchange base URL so as to check whether the port
diff --git a/src/lib/testing_api_trait_cmd.c b/src/lib/testing_api_trait_cmd.c
index 0c7dc0c86..7982f8e8d 100644
--- a/src/lib/testing_api_trait_cmd.c
+++ b/src/lib/testing_api_trait_cmd.c
@@ -40,14 +40,13 @@
* kind of traits do not need this index. For
* example, a "batch" CMD returns always the
* CMD currently being executed.
- * @param[out] cmd_ where to write the wire details.
+ * @param[out] _cmd where to write the wire details.
* @return #GNUNET_OK on success.
*/
int
-TALER_TESTING_get_trait_cmd
- (const struct TALER_TESTING_Command *cmd,
- unsigned int index,
- struct TALER_TESTING_Command **_cmd)
+TALER_TESTING_get_trait_cmd (const struct TALER_TESTING_Command *cmd,
+ unsigned int index,
+ struct TALER_TESTING_Command **_cmd)
{
return cmd->traits (cmd->cls,
(const void **) _cmd,