1 files changed, 18 insertions, 49 deletions

diff --git a/src/testing/testing_api_cmd_local_test_finished.c b/src/testing/testing_api_cmd_local_test_finished.c
index b50e471e9..0e7e214dc 100644
--- a/src/testing/testing_api_cmd_local_test_finished.c
+++ b/src/testing/testing_api_cmd_local_test_finished.c
@@ -19,7 +19,7 @@
  */
 
 /**
- * @file testing_api_cmd_block_until_all_peers_started.c
+ * @file testing_api_cmd_local_test_finished.c
  * @brief cmd to block the interpreter loop until all peers started.
  * @author t3sserakt
  */
@@ -40,6 +40,7 @@
  */
 struct LocalFinishedState
 {
+
   /**
    * Callback to write messages to the master loop.
    *
@@ -55,26 +56,11 @@ struct LocalFinishedState
 
 
 /**
- * Trait function of this cmd does nothing.
- *
- */
-static int
-local_test_finished_traits (void *cls,
-                            const void **ret,
-                            const char *trait,
-                            unsigned int index)
-{
-  return GNUNET_OK;
-}
-
-
-/**
  * The cleanup function of this cmd frees resources the cmd allocated.
  *
  */
 static void
-local_test_finished_cleanup (void *cls,
-                             const struct GNUNET_TESTING_Command *cmd)
+local_test_finished_cleanup (void *cls)
 {
   struct LocalFinishedState *lfs = cls;
 
@@ -89,11 +75,9 @@ local_test_finished_cleanup (void *cls,
  */
 static void
 local_test_finished_run (void *cls,
-                         const struct GNUNET_TESTING_Command *cmd,
                          struct GNUNET_TESTING_Interpreter *is)
 {
   struct LocalFinishedState *lfs = cls;
-
   struct GNUNET_CMDS_LOCAL_FINISHED *reply;
   size_t msg_length;
 
@@ -102,22 +86,8 @@ local_test_finished_run (void *cls,
   reply->header.type = htons (GNUNET_MESSAGE_TYPE_CMDS_HELPER_LOCAL_FINISHED);
   reply->header.size = htons ((uint16_t) msg_length);
   lfs->reply = reply;
-  lfs->write_message ((struct GNUNET_MessageHeader *) reply, msg_length);
-}
-
-
-/**
- * This finish function will stop the local loop without shutting down the scheduler, because we do not call the continuation, which is the interpreter_next method.
- *
- */
-static int
-local_test_finished_finish (void *cls,
-                            GNUNET_SCHEDULER_TaskCallback cont,
-                            void *cont_cls)
-{
-  LOG (GNUNET_ERROR_TYPE_ERROR,
-       "Stopping local loop\n");
-  return GNUNET_YES;
+  lfs->write_message ((struct GNUNET_MessageHeader *) reply,
+                      msg_length);
 }
 
 
@@ -129,23 +99,22 @@ local_test_finished_finish (void *cls,
  * @return command.
  */
 struct GNUNET_TESTING_Command
-GNUNET_TESTING_cmd_local_test_finished (const char *label,
-                                        TESTING_CMD_HELPER_write_cb
-                                        write_message)
+GNUNET_TESTING_cmd_local_test_finished (
+  const char *label,
+  TESTING_CMD_HELPER_write_cb write_message)
 {
   struct LocalFinishedState *lfs;
 
   lfs = GNUNET_new (struct LocalFinishedState);
   lfs->write_message = write_message;
-
-  struct GNUNET_TESTING_Command cmd = {
-    .cls = lfs,
-    .label = label,
-    .run = &local_test_finished_run,
-    .finish = &local_test_finished_finish,
-    .cleanup = &local_test_finished_cleanup,
-    .traits = &local_test_finished_traits
-  };
-
-  return cmd;
+  {
+    struct GNUNET_TESTING_Command cmd = {
+      .cls = lfs,
+      .label = label,
+      .run = &local_test_finished_run,
+      .cleanup = &local_test_finished_cleanup,
+    };
+
+    return cmd;
+  }
 }