[totem-pl-parser] plparse: Move signal emission to separate function

[Bastien Nocera <hadess src gnome org>]

commit 5ab0f92bca40c019774bbd6183eee0f3e58ea775
Author: Bastien Nocera <hadess hadess net>
Date:   Tue Jun 25 16:42:11 2013 +0200

    plparse: Move signal emission to separate function
    
    https://bugzilla.gnome.org/show_bug.cgi?id=700768

 plparse/totem-pl-parser.c |   46 ++++++++++++++++++++++++++++----------------
 1 files changed, 29 insertions(+), 17 deletions(-)
---
diff --git a/plparse/totem-pl-parser.c b/plparse/totem-pl-parser.c
index 64be427..8214633 100644
--- a/plparse/totem-pl-parser.c
+++ b/plparse/totem-pl-parser.c
@@ -1319,6 +1319,31 @@ emit_entry_parsed_signal (EntryParsedSignalData *data)
 }
 
 static void
+totem_pl_parser_add_hash_table (TotemPlParser *parser,
+                               GHashTable    *metadata,
+                               const char    *uri,
+                               gboolean       is_playlist)
+{
+       if (g_hash_table_size (metadata) > 0 || uri != NULL) {
+               EntryParsedSignalData *data;
+
+               /* Make sure to emit the signals asynchronously, as we could be in the main loop
+                * *or* a worker thread at this point. */
+               data = g_new (EntryParsedSignalData, 1);
+               data->parser = g_object_ref (parser);
+               data->uri = g_strdup (uri);
+               data->metadata = g_hash_table_ref (metadata);
+
+               if (is_playlist == FALSE)
+                       data->signal_id = totem_pl_parser_table_signals[ENTRY_PARSED];
+               else
+                       data->signal_id = totem_pl_parser_table_signals[PLAYLIST_STARTED];
+
+               CALL_ASYNC (parser, emit_entry_parsed_signal, data);
+       }
+}
+
+static void
 totem_pl_parser_add_uri_valist (TotemPlParser *parser,
                                const gchar *first_property_name,
                                va_list      var_args)
@@ -1429,23 +1454,10 @@ totem_pl_parser_add_uri_valist (TotemPlParser *parser,
                //FIXME fix this! 396710
        }
 
-       if (g_hash_table_size (metadata) > 0 || uri != NULL) {
-               EntryParsedSignalData *data;
-
-               /* Make sure to emit the signals asynchronously, as we could be in the main loop
-                * *or* a worker thread at this point. */
-               data = g_new (EntryParsedSignalData, 1);
-               data->parser = g_object_ref (parser);
-               data->uri = g_strdup (uri);
-               data->metadata = g_hash_table_ref (metadata);
-
-               if (is_playlist == FALSE)
-                       data->signal_id = totem_pl_parser_table_signals[ENTRY_PARSED];
-               else
-                       data->signal_id = totem_pl_parser_table_signals[PLAYLIST_STARTED];
-
-               CALL_ASYNC (parser, emit_entry_parsed_signal, data);
-       }
+       totem_pl_parser_add_hash_table (parser,
+                                       metadata,
+                                       uri,
+                                       is_playlist);
 
        g_hash_table_unref (metadata);