[ostree] fetcher: Remove message_to_request table



commit df4865e395b8c2673e2840843b2e3d9cf566d6d3
Author: Matthew Barnes <mbarnes redhat com>
Date:   Wed Sep 23 13:24:41 2015 -0400

    fetcher: Remove message_to_request table
    
    Does not appear to be needed, no lookups on the table.

 src/libostree/ostree-fetcher.c |   15 +--------------
 1 files changed, 1 insertions(+), 14 deletions(-)
---
diff --git a/src/libostree/ostree-fetcher.c b/src/libostree/ostree-fetcher.c
index 37e0f99..0a6e385 100644
--- a/src/libostree/ostree-fetcher.c
+++ b/src/libostree/ostree-fetcher.c
@@ -103,7 +103,6 @@ struct OstreeFetcher
 
   GHashTable *sending_messages; /*  SoupMessage */
 
-  GHashTable *message_to_request; /* SoupMessage -> SoupRequest */
   GHashTable *output_stream_set; /* set<GOutputStream> */
   
   guint64 total_downloaded;
@@ -128,7 +127,6 @@ _ostree_fetcher_finalize (GObject *object)
   g_clear_object (&self->client_cert);
 
   g_hash_table_destroy (self->sending_messages);
-  g_hash_table_destroy (self->message_to_request);
   g_hash_table_destroy (self->output_stream_set);
 
   g_queue_clear (&self->pending_queue);
@@ -162,7 +160,6 @@ on_request_unqueued (SoupSession  *session,
 {
   OstreeFetcher *self = user_data;
   g_hash_table_remove (self->sending_messages, msg);
-  g_hash_table_remove (self->message_to_request, msg);
 }
 
 static void
@@ -208,8 +205,6 @@ _ostree_fetcher_init (OstreeFetcher *self)
   
   self->sending_messages = g_hash_table_new_full (NULL, NULL, NULL,
                                                   (GDestroyNotify)g_object_unref);
-  self->message_to_request = g_hash_table_new_full (NULL, NULL, (GDestroyNotify)g_object_unref,
-                                                    (GDestroyNotify)pending_uri_free);
   self->output_stream_set = g_hash_table_new_full (NULL, NULL, NULL, (GDestroyNotify)g_object_unref);
 }
 
@@ -564,12 +559,6 @@ ostree_fetcher_request_uri_internal (OstreeFetcher         *self,
 
   if (is_stream)
     {
-      if (SOUP_IS_REQUEST_HTTP (pending->request))
-        {
-          g_hash_table_insert (self->message_to_request,
-                               soup_request_http_get_message ((SoupRequestHTTP*)pending->request),
-                               pending);
-        }
       soup_request_send_async (pending->request, cancellable,
                                on_request_sent, pending);
     }
@@ -597,12 +586,10 @@ ostree_fetcher_request_uri_internal (OstreeFetcher         *self,
 
       if (SOUP_IS_REQUEST_HTTP (pending->request))
         {
-          SoupMessage *msg;
+          glnx_unref_object SoupMessage *msg = NULL;
           msg = soup_request_http_get_message ((SoupRequestHTTP*) pending->request);
           if (exists && stbuf.st_size > 0)
             soup_message_headers_set_range (msg->request_headers, stbuf.st_size, -1);
-          /* Transfer ownership */
-          g_hash_table_insert (self->message_to_request, msg, pending);
         }
       pending->out_tmpfile = tmpfile;
       tmpfile = NULL; /* Transfer ownership */


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]