X-Git-Url: https://git.dogcows.com/gitweb?a=blobdiff_plain;f=openbox%2Fsession.c;h=01bb56e2fa229540388850e2fb974788ba5bb28a;hb=6372bd7c0d68793c2371ce7b6c1aa8017f913aee;hp=23ad02f66b087da8a4be15ad3a50c9510f4ba688;hpb=f189b02f857eb5cc80ca925ccb4b0e97c3614bcd;p=chaz%2Fopenbox

diff --git a/openbox/session.c b/openbox/session.c
index 23ad02f6..01bb56e2 100644
--- a/openbox/session.c
+++ b/openbox/session.c
@@ -25,8 +25,8 @@
 
 GList *session_saved_state;
 
-void session_startup(gint *argc, gchar ***argv) {}
-void session_shutdown() {}
+void session_startup(gint argc, gchar **argv) {}
+void session_shutdown(gboolean permanent) {}
 GList* session_state_find(ObClient *c) { return NULL; }
 gboolean session_state_cmp(ObSessionState *s, ObClient *c) { return FALSE; }
 void session_state_free(ObSessionState *state) {}
@@ -105,27 +105,39 @@ static void save_commands()
     g_free(prop_cmd.vals);
 }
 
-static void parse_args(gint argc, gchar **argv)
+static void remove_args(gint *argc, gchar ***argv, gint index, gint num)
 {
     gint i;
 
-    for (i = 1; i < argc; ++i) {
-        if (!strcmp((argv)[i], "--sm-client-id")) {
-            if (i == argc - 1) /* no args left */
+    for (i = index; i < index + num; ++i)
+        (*argv)[i] = (*argv)[i+num];
+    *argc -= num;
+}
+
+static void parse_args(gint *argc, gchar ***argv)
+{
+    gint i;
+
+    for (i = 1; i < *argc; ++i) {
+        if (!strcmp((*argv)[i], "--sm-client-id")) {
+            if (i == *argc - 1) /* no args left */
                 g_printerr(_("--sm-client-id requires an argument\n"));
             else {
-                sm_id = g_strdup((argv)[i+1]);
+                sm_id = g_strdup((*argv)[i+1]);
+                remove_args(argc, argv, i, 2);
                 ++i;
             }
-        } else if (!strcmp((argv)[i], "--sm-save-file")) {
-            if (i == argc - 1) /* no args left */
+        } else if (!strcmp((*argv)[i], "--sm-save-file")) {
+            if (i == *argc - 1) /* no args left */
                 g_printerr(_("--sm-save-file requires an argument\n"));
             else {
-                save_file = g_strdup((argv)[i+1]);
+                save_file = g_strdup((*argv)[i+1]);
+                remove_args(argc, argv, i, 2);
                 ++i;
             }
-        } else if (!strcmp((argv)[i], "--sm-disable")) {
+        } else if (!strcmp((*argv)[i], "--sm-disable")) {
             sm_disable = TRUE;
+            remove_args(argc, argv, i, 1);
         }
     }
 }
@@ -136,11 +148,19 @@ void session_startup(gint argc, gchar **argv)
 
     SmcCallbacks cb;
     gchar sm_err[SM_ERR_LEN];
+    gint i;
 
-    parse_args(argc, argv);
+    sm_argc = argc;
+    sm_argv = g_new(gchar*, argc);
+    for (i = 0; i < argc; ++i)
+        sm_argv[i] = argv[i];
 
-    if (sm_disable)
+    parse_args(&sm_argc, &sm_argv);
+
+    if (sm_disable) {
+        g_free(sm_argv);
         return;
+    }
 
     sm_sessions_path = g_build_filename(parse_xdg_data_home_path(),
                                         "openbox", "sessions", NULL);
@@ -162,9 +182,6 @@ void session_startup(gint argc, gchar **argv)
         g_free(filename);
     }
 
-    sm_argc = argc;
-    sm_argv = argv;
-
     cb.save_yourself.callback = sm_save_yourself;
     cb.save_yourself.client_data = NULL;
 
@@ -240,28 +257,36 @@ void session_startup(gint argc, gchar **argv)
     }
 }
 
-void session_shutdown()
+void session_shutdown(gboolean permanent)
 {
+    if (sm_disable)
+        return;
+
     g_free(sm_sessions_path);
     g_free(save_file);
     g_free(sm_id);
+    g_free(sm_argv);
 
     if (sm_conn) {
-        SmPropValue val_hint;
-        SmProp prop_hint = { SmRestartStyleHint, SmCARD8, 1, };
-        SmProp *props[1];
-        gulong hint;
+        /* if permanent is true then we will change our session state so that
+           the SM won't run us again */
+        if (permanent) {
+            SmPropValue val_hint;
+            SmProp prop_hint = { SmRestartStyleHint, SmCARD8, 1, };
+            SmProp *props[1];
+            gulong hint;
 
-        /* when we exit, we want to reset this to a more friendly state */
-        hint = SmRestartIfRunning;
-        val_hint.value = &hint;
-        val_hint.length = 1;
+            /* when we exit, we want to reset this to a more friendly state */
+            hint = SmRestartIfRunning;
+            val_hint.value = &hint;
+            val_hint.length = 1;
 
-        prop_hint.vals = &val_hint;
+            prop_hint.vals = &val_hint;
 
-        props[0] = &prop_hint;
+            props[0] = &prop_hint;
 
-        SmcSetProperties(sm_conn, 1, props);
+            SmcSetProperties(sm_conn, 1, props);
+        }
 
         SmcCloseConnection(sm_conn, 0, NULL);