[PATCH 4/7] make some function static

[Nikola Pajkovsky]

Signed-off-by: Nikola Pajkovsky <npajkovs(a)redhat.com&gt;
---
 src/applet/CCApplet.cpp |  329 ++++++++++++++++++++++++-----------------------
 src/applet/CCApplet.h   |   15 --
 2 files changed, 165 insertions(+), 179 deletions(-)

diff --git a/src/applet/CCApplet.cpp b/src/applet/CCApplet.cpp
index d27ce28..0e292bb 100644
--- a/src/applet/CCApplet.cpp
+++ b/src/applet/CCApplet.cpp
@@ -22,6 +22,149 @@
 #include "abrtlib.h"
 #include "CCApplet.h"
 
+static bool load_icons(struct applet *applet)
+{
+    //FIXME: just a tmp workaround
+    return false;
+    int stage;
+    for (stage = ICON_DEFAULT; stage < ICON_STAGE_LAST; stage++)
+    {
+        char name[sizeof(ICON_DIR"/abrt%02d.png")];
+        GError *error = NULL;
+        if (snprintf(name, sizeof(ICON_DIR"/abrt%02d.png"),
ICON_DIR"/abrt%02d.png", stage) > 0)
+        {
+            applet->icon_stages_buff[stage] = gdk_pixbuf_new_from_file(name,
&error);
+            if (error != NULL)
+            {
+                error_msg("Can't load pixbuf from %s, animation is
disabled", name);
+                return false;
+            }
+        }
+    }
+    return true;
+}
+
+static void stop_animate_icon(struct applet *applet)
+{
+    /* animator should be 0 if icons are not loaded, so this should be safe */
+    if (applet->animator != 0)
+    {
+        g_source_remove(applet->animator);
+        gtk_status_icon_set_from_pixbuf(applet->status_icon,
+                                        applet->icon_stages_buff[ICON_DEFAULT]
+        );
+        applet->animator = 0;
+    }
+}
+
+//this action should open the reporter dialog directly, without showing the main window
+static void action_report(NotifyNotification *notification, gchar *action, gpointer
user_data)
+{
+    struct applet *applet = (struct applet *)user_data;
+    if (applet->daemon_running)
+    {
+        pid_t pid = vfork();
+        if (pid < 0)
+            perror_msg("vfork");
+        if (pid == 0)
+        { /* child */
+            char *buf = xasprintf("--report=%s", applet->last_crash_id);
+            signal(SIGCHLD, SIG_DFL); /* undo SIG_IGN in abrt-applet */
+            execl(BIN_DIR"/abrt-gui", "abrt-gui", buf, (char*)
NULL);
+            /* Did not find abrt-gui in installation directory. Oh well */
+            /* Trying to find it in PATH */
+            execlp("abrt-gui", "abrt-gui", buf, (char*) NULL);
+            perror_msg_and_die("Can't execute abrt-gui");
+        }
+        GError *err = NULL;
+        notify_notification_close(notification, &err);
+        if (err != NULL)
+        {
+            error_msg("%s", err->message);
+            g_error_free(err);
+        }
+        gtk_status_icon_set_visible(applet->status_icon, false);
+        stop_animate_icon(applet);
+    }
+}
+
+//this action should open the main window
+static void action_open_gui(NotifyNotification *notification, gchar *action, gpointer
user_data)
+{
+    struct applet *applet = (struct applet*)user_data;
+    if (applet->daemon_running)
+    {
+        pid_t pid = vfork();
+        if (pid < 0)
+            perror_msg("vfork");
+        if (pid == 0)
+        { /* child */
+            signal(SIGCHLD, SIG_DFL); /* undo SIG_IGN in abrt-applet */
+            execl(BIN_DIR"/abrt-gui", "abrt-gui", (char*) NULL);
+            /* Did not find abrt-gui in installation directory. Oh well */
+            /* Trying to find it in PATH */
+            execlp("abrt-gui", "abrt-gui", (char*) NULL);
+            perror_msg_and_die("Can't execute abrt-gui");
+        }
+        GError *err = NULL;
+        notify_notification_close(notification, &err);
+        if (err != NULL)
+        {
+            error_msg("%s", err->message);
+            g_error_free(err);
+        }
+        gtk_status_icon_set_visible(applet->status_icon, false);
+        stop_animate_icon(applet);
+    }
+}
+
+static void on_menu_popup_cb(GtkStatusIcon *status_icon,
+                        guint          button,
+                        guint          activate_time,
+                        gpointer       user_data)
+{
+    struct applet *applet = (struct applet*)user_data;
+    /* stop the animation */
+    stop_animate_icon(applet);
+
+    if (applet->menu != NULL)
+    {
+        gtk_menu_popup(GTK_MENU(applet->menu),
+                NULL, NULL,
+                gtk_status_icon_position_menu,
+                status_icon, button, activate_time);
+    }
+}
+
+// why it is not named with suffix _cb when it is callback for g_timeout_add?
+static gboolean update_icon(void *user_data)
+{
+    struct applet *applet = (struct applet*)user_data;
+    if (applet->status_icon && applet->animation_stage <
ICON_STAGE_LAST)
+    {
+        gtk_status_icon_set_from_pixbuf(applet->status_icon,
+                                       
applet->icon_stages_buff[applet->animation_stage++]);
+    }
+    if (applet->animation_stage == ICON_STAGE_LAST)
+    {
+        applet->animation_stage = 0;
+    }
+    if (--applet->anim_countdown == 0)
+    {
+        stop_animate_icon(applet);
+    }
+    return true;
+}
+
+static void animate_icon(struct applet* applet)
+{
+    if (applet->animator == 0)
+    {
+        applet->animator = g_timeout_add(100, update_icon, applet);
+        applet->anim_countdown = 10 * 3; /* 3 sec */
+    }
+}
+
 static void on_notify_close(NotifyNotification *notification, gpointer user_data)
 {
     g_object_unref(notification);
@@ -126,6 +269,28 @@ static GtkWidget *create_menu(struct applet *applet)
     return menu;
 }
 
+static void on_applet_activate_cb(GtkStatusIcon *status_icon, gpointer user_data)
+{
+    struct applet *applet = (struct applet*)user_data;
+    if (applet->daemon_running)
+    {
+        pid_t pid = vfork();
+        if (pid < 0)
+            perror_msg("vfork");
+        if (pid == 0)
+        { /* child */
+            signal(SIGCHLD, SIG_DFL); /* undo SIG_IGN in abrt-applet */
+            execl(BIN_DIR"/abrt-gui", "abrt-gui", (char*) NULL);
+            /* Did not find abrt-gui in installation directory. Oh well */
+            /* Trying to find it in PATH */
+            execlp("abrt-gui", "abrt-gui", (char*) NULL);
+            perror_msg_and_die("Can't execute abrt-gui");
+        }
+        gtk_status_icon_set_visible(applet->status_icon, false);
+        stop_animate_icon(applet);
+    }
+}
+
 struct applet *applet_new(const char* app_name)
 {
     struct applet *applet = (struct applet*)xmalloc(sizeof(struct applet));
@@ -181,65 +346,6 @@ void set_icon_tooltip(struct applet *applet, const char *format,
...)
     free(buf);
 }
 
-void action_report(NotifyNotification *notification, gchar *action, gpointer user_data)
-{
-    struct applet *applet = (struct applet *)user_data;
-    if (applet->daemon_running)
-    {
-        pid_t pid = vfork();
-        if (pid < 0)
-            perror_msg("vfork");
-        if (pid == 0)
-        { /* child */
-            char *buf = xasprintf("--report=%s", applet->last_crash_id);
-            signal(SIGCHLD, SIG_DFL); /* undo SIG_IGN in abrt-applet */
-            execl(BIN_DIR"/abrt-gui", "abrt-gui", buf, (char*)
NULL);
-            /* Did not find abrt-gui in installation directory. Oh well */
-            /* Trying to find it in PATH */
-            execlp("abrt-gui", "abrt-gui", buf, (char*) NULL);
-            perror_msg_and_die("Can't execute abrt-gui");
-        }
-        GError *err = NULL;
-        notify_notification_close(notification, &err);
-        if (err != NULL)
-        {
-            error_msg("%s", err->message);
-            g_error_free(err);
-        }
-        gtk_status_icon_set_visible(applet->status_icon, false);
-        stop_animate_icon(applet);
-    }
-}
-
-void action_open_gui(NotifyNotification *notification, gchar *action, gpointer
user_data)
-{
-    struct applet *applet = (struct applet*)user_data;
-    if (applet->daemon_running)
-    {
-        pid_t pid = vfork();
-        if (pid < 0)
-            perror_msg("vfork");
-        if (pid == 0)
-        { /* child */
-            signal(SIGCHLD, SIG_DFL); /* undo SIG_IGN in abrt-applet */
-            execl(BIN_DIR"/abrt-gui", "abrt-gui", (char*) NULL);
-            /* Did not find abrt-gui in installation directory. Oh well */
-            /* Trying to find it in PATH */
-            execlp("abrt-gui", "abrt-gui", (char*) NULL);
-            perror_msg_and_die("Can't execute abrt-gui");
-        }
-        GError *err = NULL;
-        notify_notification_close(notification, &err);
-        if (err != NULL)
-        {
-            error_msg("%s", err->message);
-            g_error_free(err);
-        }
-        gtk_status_icon_set_visible(applet->status_icon, false);
-        stop_animate_icon(applet);
-    }
-}
-
 void crash_notify(struct applet *applet, const char* crash_id, const char *format, ...)
 {
     applet->last_crash_id = crash_id;
@@ -294,46 +400,6 @@ void message_notify(struct applet *applet, const char *format, ...)
     }
 }
 
-void on_applet_activate_cb(GtkStatusIcon *status_icon, gpointer user_data)
-{
-    struct applet *applet = (struct applet*)user_data;
-    if (applet->daemon_running)
-    {
-        pid_t pid = vfork();
-        if (pid < 0)
-            perror_msg("vfork");
-        if (pid == 0)
-        { /* child */
-            signal(SIGCHLD, SIG_DFL); /* undo SIG_IGN in abrt-applet */
-            execl(BIN_DIR"/abrt-gui", "abrt-gui", (char*) NULL);
-            /* Did not find abrt-gui in installation directory. Oh well */
-            /* Trying to find it in PATH */
-            execlp("abrt-gui", "abrt-gui", (char*) NULL);
-            perror_msg_and_die("Can't execute abrt-gui");
-        }
-        gtk_status_icon_set_visible(applet->status_icon, false);
-        stop_animate_icon(applet);
-    }
-}
-
-void on_menu_popup_cb(GtkStatusIcon *status_icon,
-                        guint          button,
-                        guint          activate_time,
-                        gpointer       user_data)
-{
-    struct applet *applet = (struct applet*)user_data;
-    /* stop the animation */
-    stop_animate_icon(applet);
-
-    if (applet->menu != NULL)
-    {
-        gtk_menu_popup(GTK_MENU(applet->menu),
-                NULL, NULL,
-                gtk_status_icon_position_menu,
-                status_icon, button, activate_time);
-    }
-}
-
 void show_icon(struct applet *applet)
 {
     gtk_status_icon_set_visible(applet->status_icon, true);
@@ -379,71 +445,6 @@ void enable(struct applet *applet, const char *reason)
     show_icon(applet);
 }
 
-// why it is not named with suffix _cb when it is callback for g_timeout_add?
-gboolean update_icon(void *user_data)
-{
-    struct applet *applet = (struct applet*)user_data;
-    if (applet->status_icon && applet->animation_stage <
ICON_STAGE_LAST)
-    {
-        gtk_status_icon_set_from_pixbuf(applet->status_icon,
-                                       
applet->icon_stages_buff[applet->animation_stage++]);
-    }
-    if (applet->animation_stage == ICON_STAGE_LAST)
-    {
-        applet->animation_stage = 0;
-    }
-    if (--applet->anim_countdown == 0)
-    {
-        stop_animate_icon(applet);
-    }
-    return true;
-}
-
-void animate_icon(struct applet* applet)
-{
-    if (applet->animator == 0)
-    {
-        applet->animator = g_timeout_add(100, update_icon, applet);
-        applet->anim_countdown = 10 * 3; /* 3 sec */
-    }
-}
-
-void stop_animate_icon(struct applet *applet)
-{
-    /* animator should be 0 if icons are not loaded, so this should be safe */
-    if (applet->animator != 0)
-    {
-        g_source_remove(applet->animator);
-        gtk_status_icon_set_from_pixbuf(applet->status_icon,
-                                        applet->icon_stages_buff[ICON_DEFAULT]
-        );
-        applet->animator = 0;
-    }
-}
-
-bool load_icons(struct applet *applet)
-{
-    //FIXME: just a tmp workaround
-    return false;
-    int stage;
-    for (stage = ICON_DEFAULT; stage < ICON_STAGE_LAST; stage++)
-    {
-        char name[sizeof(ICON_DIR"/abrt%02d.png")];
-        GError *error = NULL;
-        if (snprintf(name, sizeof(ICON_DIR"/abrt%02d.png"),
ICON_DIR"/abrt%02d.png", stage) > 0)
-        {
-            applet->icon_stages_buff[stage] = gdk_pixbuf_new_from_file(name,
&error);
-            if (error != NULL)
-            {
-                error_msg("Can't load pixbuf from %s, animation is
disabled", name);
-                return false;
-            }
-        }
-    }
-    return true;
-}
-
-
 //int CApplet::AddEvent(int pUUID, const char *pProgname)
 //{
 //    m_mapEvents[pUUID] = "pProgname";
diff --git a/src/applet/CCApplet.h b/src/applet/CCApplet.h
index a7fb543..f1949f3 100644
--- a/src/applet/CCApplet.h
+++ b/src/applet/CCApplet.h
@@ -72,19 +72,4 @@ void message_notify(struct applet *applet, const char *format, ...);
 void disable(struct applet *applet, const char *reason);
 void enable(struct applet *applet, const char *reason);
 
-// static in next patch
-void on_applet_activate_cb(GtkStatusIcon *status_icon, gpointer user_data);
-//this action should open the reporter dialog directly, without showing the main window
-void action_report(NotifyNotification *notification, gchar *action, gpointer user_data);
-//this action should open the main window
-void action_open_gui(NotifyNotification *notification, gchar *action, gpointer
user_data);
-void on_menu_popup_cb(GtkStatusIcon *status_icon,
-                    guint          button,
-                    guint          activate_time,
-                    gpointer       user_data);
-gboolean update_icon(void *data);
-void animate_icon(struct applet *applet);
-void stop_animate_icon(struct applet *applet);
-bool load_icons(struct applet *applet);
-
 #endif
-- 
1.7.1