[gnome-software] trivial: Don't profile gs_plugin_refine_app() as it adds too much overhead



commit 9cbf05d7bddc5babd1f13d54f1c5a9d258be416b
Author: Richard Hughes <richard hughsie com>
Date:   Tue Nov 29 16:15:16 2016 +0000

    trivial: Don't profile gs_plugin_refine_app() as it adds too much overhead

 src/gs-plugin-loader.c |   26 ++++++++++++++------------
 1 files changed, 14 insertions(+), 12 deletions(-)
---
diff --git a/src/gs-plugin-loader.c b/src/gs-plugin-loader.c
index 5323c83..6063373 100644
--- a/src/gs-plugin-loader.c
+++ b/src/gs-plugin-loader.c
@@ -474,19 +474,21 @@ gs_plugin_loader_call_vfunc (GsPluginLoaderJob *job,
                return TRUE;
 
        /* profile */
-       if (job->function_name_parent == NULL) {
-               ptask = as_profile_start (priv->profile,
-                                         "GsPlugin::%s(%s)",
-                                         gs_plugin_get_name (plugin),
-                                         job->function_name);
-       } else {
-               ptask = as_profile_start (priv->profile,
-                                         "GsPlugin::%s(%s;%s)",
-                                         gs_plugin_get_name (plugin),
-                                         job->function_name_parent,
-                                         job->function_name);
+       if (g_strcmp0 (job->function_name, "gs_plugin_refine_app") != 0) {
+               if (job->function_name_parent == NULL) {
+                       ptask = as_profile_start (priv->profile,
+                                                 "GsPlugin::%s(%s)",
+                                                 gs_plugin_get_name (plugin),
+                                                 job->function_name);
+               } else {
+                       ptask = as_profile_start (priv->profile,
+                                                 "GsPlugin::%s(%s;%s)",
+                                                 gs_plugin_get_name (plugin),
+                                                 job->function_name_parent,
+                                                 job->function_name);
+               }
+               g_assert (ptask != NULL);
        }
-       g_assert (ptask != NULL);
 
        /* fallback if unset */
        if (app == NULL)


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