[tracker: 14/30] libtracker-miner: Use g_slice_new0() instead of g_new0() where it makes sense
- From: Carlos Garnacho <carlosg src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [tracker: 14/30] libtracker-miner: Use g_slice_new0() instead of g_new0() where it makes sense
- Date: Tue, 21 Jan 2014 12:00:16 +0000 (UTC)
commit 68471ab2b393b13638e062a5c9088ea8d31b6699
Author: Martyn Russell <martyn lanedo com>
Date: Wed Jan 15 16:25:51 2014 +0000
libtracker-miner: Use g_slice_new0() instead of g_new0() where it makes sense
Given we're reusing these struct sizes many times, it makes sense to use
GSlice.
src/libtracker-miner/tracker-decorator.c | 18 ++++++++++++++----
1 files changed, 14 insertions(+), 4 deletions(-)
---
diff --git a/src/libtracker-miner/tracker-decorator.c b/src/libtracker-miner/tracker-decorator.c
index 113eb4b..d2e4de7 100644
--- a/src/libtracker-miner/tracker-decorator.c
+++ b/src/libtracker-miner/tracker-decorator.c
@@ -101,7 +101,7 @@ tracker_decorator_info_new (TrackerSparqlCursor *cursor)
{
TrackerDecoratorInfo *info;
- info = g_new0 (TrackerDecoratorInfo, 1);
+ info = g_slice_new0 (TrackerDecoratorInfo);
info->urn = g_strdup (tracker_sparql_cursor_get_string (cursor, 0, NULL));
info->url = g_strdup (tracker_sparql_cursor_get_string (cursor, 2, NULL));
info->mimetype = g_strdup (tracker_sparql_cursor_get_string (cursor, 3, NULL));
@@ -128,7 +128,7 @@ tracker_decorator_info_unref (TrackerDecoratorInfo *info)
g_free (info->urn);
g_free (info->url);
g_free (info->mimetype);
- g_free (info);
+ g_slice_free (TrackerDecoratorInfo, info);
}
static TaskData *
@@ -138,7 +138,7 @@ task_data_new (TrackerDecorator *decorator,
{
TaskData *data;
- data = g_new0 (TaskData, 1);
+ data = g_slice_new0 (TaskData);
data->decorator = decorator;
data->first = first;
data->last = last;
@@ -147,6 +147,16 @@ task_data_new (TrackerDecorator *decorator,
}
static void
+task_data_free (TaskData *data)
+{
+ if (!data) {
+ return;
+ }
+
+ g_slice_free (TaskData, data);
+}
+
+static void
decorator_update_state (TrackerDecorator *decorator,
const gchar *message,
gboolean estimate_time)
@@ -1114,7 +1124,7 @@ query_next_items (TrackerDecorator *decorator,
"}", id_string->str, priv->data_source);
data = task_data_new (decorator, priv->elem_queue->head, last);
- g_task_set_task_data (task, data, (GDestroyNotify) g_free);
+ g_task_set_task_data (task, data, (GDestroyNotify) task_data_free);
tracker_sparql_connection_query_async (sparql_conn, query,
g_task_get_cancellable (task),
query_next_items_cb, task);
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]