1 files changed, 39 insertions, 41 deletions

diff --git a/src/fs/gnunet-fs-gtk_main-window-namespace-dropdown.c b/src/fs/gnunet-fs-gtk_main-window-namespace-dropdown.c
index 7e2df958..666f58bf 100644
--- a/src/fs/gnunet-fs-gtk_main-window-namespace-dropdown.c
+++ b/src/fs/gnunet-fs-gtk_main-window-namespace-dropdown.c
@@ -1,6 +1,6 @@
 /*
      This file is part of GNUnet
-     (C) 2011, 2012 Christian Grothoff (and other contributing authors)
+     (C) 2011-2013 Christian Grothoff (and other contributing authors)
 
      GNUnet is free software; you can redistribute it and/or modify
      it under the terms of the GNU General Public License as published
@@ -301,7 +301,8 @@ add_namespace_to_ts (void *cls,
 {
   GtkTreeStore *ts = cls;
   char *root;
-  char *ns_name, *unique_ns_name;
+  char *ns_name;
+  char *unique_ns_name;
   struct GNUNET_CRYPTO_EccPublicKey *nsid;
   char *description;
   int desc_is_a_dup;
@@ -316,8 +317,8 @@ add_namespace_to_ts (void *cls,
   GNUNET_FS_pseudonym_get_info (GNUNET_FS_GTK_get_configuration (),
                                 pseudonym, NULL, NULL, 
                                 &ns_name, NULL);
-  unique_ns_name = GNUNET_FS_pseudonym_name_uniquify (
-      GNUNET_FS_GTK_get_configuration (), pseudonym, ns_name, NULL);
+  unique_ns_name = GNUNET_FS_pseudonym_name_uniquify (GNUNET_FS_GTK_get_configuration (),
+                                                      pseudonym, ns_name, NULL);
   GNUNET_free (ns_name);
   nsid = GNUNET_new (struct GNUNET_CRYPTO_EccPublicKey);
   *nsid = *pseudonym;
@@ -355,27 +356,22 @@ GNUNET_GTK_main_window_refresh_ns_list (struct GNUNET_GTK_MainWindowContext *mai
 {
   GtkTreeIter iter;
   GtkTreePath *treepath;
-  struct GNUNET_CRYPTO_EccPublicKey *key = NULL;
+  GtkTreeModel *model;
+  struct GNUNET_CRYPTO_EccPublicKey *key;
   struct GNUNET_CRYPTO_EccPublicKey *selected_ns_id;
-
-  gboolean found = FALSE;
-  gchar *value = NULL;
+  gboolean found;
+  gchar *value;
     
+  key = NULL;
   if (NULL != main_ctx->selected_ns_row)
   {
-    GtkTreeModel *model;
     treepath = gtk_tree_row_reference_get_path (main_ctx->selected_ns_row);
     model = gtk_tree_view_get_model (main_ctx->ns_selector_treeview);
-    if (model)
-    {
-      if (gtk_tree_model_get_iter (model, &iter, treepath))
-      {
-        gtk_tree_model_get (model, &iter,
-                            GNUNET_GTK_FS_MAIN_WINDOW_SEARCH_NAMESPACE_MC_KEY,
-                            &key,
-                            -1);
-      }
-    }
+    if (gtk_tree_model_get_iter (model, &iter, treepath))
+      gtk_tree_model_get (model, &iter,
+                          GNUNET_GTK_FS_MAIN_WINDOW_SEARCH_NAMESPACE_MC_KEY,
+                          &key,
+                          -1);
     gtk_tree_path_free (treepath);
     gtk_tree_row_reference_free (main_ctx->selected_ns_row);
     main_ctx->selected_ns_row = NULL;
@@ -387,23 +383,20 @@ GNUNET_GTK_main_window_refresh_ns_list (struct GNUNET_GTK_MainWindowContext *mai
     *selected_ns_id = *key;
   }
 
-  if (gtk_tree_model_get_iter_first (GTK_TREE_MODEL (
-      main_ctx->search_ns_treestore), &iter))
+  if (gtk_tree_model_get_iter_first (GTK_TREE_MODEL (main_ctx->search_ns_treestore),
+                                     &iter))
   {
-    while (TRUE)
+    do 
     {
       gtk_tree_model_get (GTK_TREE_MODEL (main_ctx->search_ns_treestore), &iter,
                           GNUNET_GTK_FS_MAIN_WINDOW_SEARCH_NAMESPACE_MC_KEY,
                           &key,
                           -1);
       GNUNET_free_non_null (key);
-      if (TRUE != gtk_tree_model_iter_next (GTK_TREE_MODEL (
-        main_ctx->search_ns_treestore), &iter))
-        break;
-    }
+    } while (gtk_tree_model_iter_next (GTK_TREE_MODEL (main_ctx->search_ns_treestore), 
+                                       &iter));
   }
   gtk_tree_store_clear (main_ctx->search_ns_treestore);
- 
   gtk_tree_store_insert_with_values (main_ctx->search_ns_treestore, &iter, NULL, G_MAXINT,
                                      GNUNET_GTK_FS_MAIN_WINDOW_SEARCH_NAMESPACE_MC_NAME,
                                      "Any",
@@ -417,11 +410,15 @@ GNUNET_GTK_main_window_refresh_ns_list (struct GNUNET_GTK_MainWindowContext *mai
 
   if (NULL != main_ctx->ns_discovery_handle)
     GNUNET_FS_pseudonym_discovery_callback_unregister (main_ctx->ns_discovery_handle);
-  main_ctx->ns_discovery_handle = GNUNET_FS_pseudonym_discovery_callback_register (main_ctx->cfg,
-      add_namespace_to_ts, main_ctx->search_ns_treestore);
-
-  if (gtk_tree_model_get_iter_first (GTK_TREE_MODEL (
-      main_ctx->search_ns_treestore), &iter))
+  main_ctx->ns_discovery_handle 
+    = GNUNET_FS_pseudonym_discovery_callback_register (main_ctx->cfg,
+                                                       &add_namespace_to_ts,
+                                                       main_ctx->search_ns_treestore);
+
+  found = FALSE;
+  value = NULL;
+  if (gtk_tree_model_get_iter_first (GTK_TREE_MODEL (main_ctx->search_ns_treestore), 
+                                     &iter))
   {
     while (TRUE)
     {
@@ -431,20 +428,21 @@ GNUNET_GTK_main_window_refresh_ns_list (struct GNUNET_GTK_MainWindowContext *mai
                           GNUNET_GTK_FS_MAIN_WINDOW_SEARCH_NAMESPACE_MC_KEY,
                           &key,
                           -1);
-      if (selected_ns_id == NULL)
+      if (NULL == selected_ns_id)
         found = TRUE;
-      else if (key != NULL && memcmp (key, selected_ns_id, sizeof (struct GNUNET_CRYPTO_EccPublicKey)) == 0)
+      if ( (key != NULL) && 
+           (0 == memcmp (key, selected_ns_id, sizeof (struct GNUNET_CRYPTO_EccPublicKey))) )
         found = TRUE;
-      if (found || (TRUE != gtk_tree_model_iter_next (GTK_TREE_MODEL (
-        main_ctx->search_ns_treestore), &iter)))
+      if (found || 
+          (! gtk_tree_model_iter_next (GTK_TREE_MODEL (main_ctx->search_ns_treestore), 
+                                       &iter)))
         break;
-      else
-        g_free (value);
+      g_free (value);
     }
   }
   if ( (!found) &&
-       gtk_tree_model_get_iter_first (GTK_TREE_MODEL (
-                                                      main_ctx->search_ns_treestore), &iter))
+       gtk_tree_model_get_iter_first (GTK_TREE_MODEL (main_ctx->search_ns_treestore),
+                                      &iter))
   {
     gtk_tree_model_get (GTK_TREE_MODEL (main_ctx->search_ns_treestore), &iter,
                         GNUNET_GTK_FS_MAIN_WINDOW_SEARCH_NAMESPACE_MC_NAME,
@@ -460,7 +458,7 @@ GNUNET_GTK_main_window_refresh_ns_list (struct GNUNET_GTK_MainWindowContext *mai
   if (value != NULL)
   {
     gtk_label_set_text (main_ctx->search_ns_label, value);
-    g_free(value);
+    g_free (value);
   }
   GNUNET_free_non_null (selected_ns_id);
 }