mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2024-07-03 13:16:04 +00:00
Code Issues 1 Projects Releases Wiki Activity

uri: Build doubly-linked list by prepending items

Browse source 
As outlined in the API documentation, g_list_append() iterates over the whole
list, which can quickly introduce performance issues when the list becomes very
big, such as for data URIs for instance.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1909>
This commit is contained in:
Philippe Normand 2022-03-10 18:22:49 +00:00 committed by GStreamer Marge Bot
parent b8ccf7f802
commit 4277af3219
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
subprojects/gstreamer/gst/gsturi.c
View file

@ -1315,7 +1315,7 @@ _gst_uri_string_to_list (const gchar * str, const gchar * sep, gboolean convert,
for (next_elem = split_str; *next_elem; next_elem += 1) {
gchar *elem = *next_elem;
if (*elem == '\0') {
new_list = g_list_append (new_list, NULL);
new_list = g_list_prepend (new_list, NULL);
} else {
if (convert && !unescape) {
gchar *next_sep;
@ -1331,7 +1331,7 @@ _gst_uri_string_to_list (const gchar * str, const gchar * sep, gboolean convert,
g_free (elem);
elem = *next_elem;
}
new_list = g_list_append (new_list, g_strdup (elem));
new_list = g_list_prepend (new_list, g_strdup (elem));
}
}
}
@ -1340,7 +1340,7 @@ _gst_uri_string_to_list (const gchar * str, const gchar * sep, gboolean convert,
g_free (pct_sep);
}
return new_list;
return g_list_reverse (new_list);
}
static GHashTable *