some notes to self

diff --git a/openbox/client.c b/openbox/client.c
index 0cc393ba13040b3187b856fb9e12dcf6852663be..e7af04b85270e97d115683a6c3b1f3bbd71fe338 100644 (file)
--- a/openbox/client.c
+++ b/openbox/client.c
@@ -295,6 +295,7 @@ void client_manage(Window window)
     client_apply_startup_state(self);
 
     /* get and set application level settings */
+    /* XXX move that function here */
     settings = (ObAppSetting *) get_client_settings(self);
 
     if (settings) {
@@ -361,10 +362,11 @@ void client_manage(Window window)
         gint x = self->area.x, ox = x;
         gint y = self->area.y, oy = y;
 
-        place_client(self, &x, &y);
-
         if (settings)
+            /* XXX put this in place.c */
             place_window_from_settings(settings, self, &x, &y);
+        else
+            place_client(self, &x, &y);
 
         /* make sure the window is visible. */
         client_find_onscreen(self, &x, &y,

diff --git a/openbox/per_app_settings.c b/openbox/per_app_settings.c
index c8ab4ed52cf8adc756bbc30015861b4732aca613..73331cc6ff0d1774d37f2e5fc55dc42f9ddcd00b 100644 (file)
--- a/openbox/per_app_settings.c
+++ b/openbox/per_app_settings.c
@@ -20,6 +20,9 @@
 #include "screen.h"
 #include "config.h"
 
+/* XXX put in client.c */
+/* This should possibly do something more interesting than just match
+ * against WM_CLASS literally. */
 ObAppSetting *get_client_settings(ObClient *client)
 {
     GSList *a = config_per_app_settings;
@@ -38,6 +41,7 @@ ObAppSetting *get_client_settings(ObClient *client)
     return NULL;
 }
 
+/* XXX put in place.c */
 void place_window_from_settings(ObAppSetting *setting, ObClient *client, gint *x, gint *y)
 {
     gint px, py, i;