Re #142: Tidy up debugging

src/CbMedia.c

@@ -32,7 +32,6 @@ static void
 cb_media_finalize (GObject *object)
 {
   CbMedia *media = CB_MEDIA (object);
-  g_debug("MEM Finalising %s with %d references", media->url, cairo_surface_get_reference_count(media->surface));
   cairo_surface_destroy (media->surface);
   if (media->surface_hires != NULL) {
     cairo_surface_destroy (media->surface_hires);

src/CbMediaDownloader.c

@@ -92,22 +92,21 @@ load_animation (GInputStream *input_stream,
     {
       return;
     }
-  // XXX: How often does this function need to do what it says (and potentially use lots of memory temporarily, as it decodes a video until it can make a picture)?
+
   frame = gdk_pixbuf_animation_get_static_image (animation);
 
   if (g_cancellable_is_cancelled (cancellable))
     {
       g_object_unref (animation);
       return;
     }
-  g_debug("MEM Image is %s", gdk_pixbuf_animation_is_static_image (animation) ? "static" : "animated");
+
   if (!gdk_pixbuf_animation_is_static_image (animation))
     *target_animation = animation; /* Takes ref */
   else
     *target_animation = NULL;
 
   has_alpha = gdk_pixbuf_get_has_alpha (frame);
-  g_debug("MEM Creating %d×%d %d bit surface", gdk_pixbuf_get_width (frame), gdk_pixbuf_get_height (frame), has_alpha ? 32 : 24);
   surface = cairo_image_surface_create (has_alpha ? CAIRO_FORMAT_ARGB32 : CAIRO_FORMAT_RGB24,
                                         gdk_pixbuf_get_width (frame),
                                         gdk_pixbuf_get_height (frame));
@@ -168,7 +167,6 @@ load_media_url (const char *url, LoadingData *task_data,
   input_stream = g_memory_input_stream_new_from_data (msg->response_body->data,
                                                       msg->response_body->length,
                                                       NULL);
-  g_debug("MEM Loading animation for %s", url);
   load_animation (input_stream, surface, animation, cancellable, error);
   g_input_stream_close (input_stream, NULL, NULL);
   g_object_unref (input_stream);

src/CbTypes.c

@@ -430,7 +430,7 @@ cb_mini_tweet_parse_entities (CbMiniTweet *t,
                 const char *url = json_object_has_member (media_obj, "media_url_https") ?
                   json_object_get_string_member (media_obj, "media_url_https") :
                   json_object_get_string_member (media_obj, "media_url");
-                g_warning("URL %s was marked as type photo but not a media candidate");
+                g_warning("URL %s was marked as type photo but not a media candidate", url);
               }
 
 
@@ -490,7 +490,7 @@ cb_mini_tweet_parse_entities (CbMiniTweet *t,
                   t->medias[t->n_medias]->height = height;
                   t->medias[t->n_medias]->thumb_width  = thumb_width;
                   t->medias[t->n_medias]->thumb_height = thumb_height;
-                  g_debug("Video: %d×%d; Thumb: %d×%d", width, height, thumb_width, thumb_height);
+
                   if (json_object_has_member (media_obj, "ext_alt_text")) {
                     // Only "extended media" for GIFs has alt text. Videos never do.
                     t->medias[t->n_medias]->alt_text = g_strdup (json_object_get_string_member (media_obj, "ext_alt_text"));