From ea8393fdc99d1a540d30e31ecf289a6515493482 Mon Sep 17 00:00:00 2001
From: Mikael Magnusson <mikachu@gmail.com>
Date: Mon, 6 Oct 2014 19:52:14 +0200
Subject: [PATCH] Move common gravity application to screen.c

---
 openbox/menuframe.c | 32 ++------------------------------
 openbox/place.c     | 24 ++----------------------
 openbox/screen.c    | 27 +++++++++++++++++++++++++++
 openbox/screen.h    |  5 +++++
 4 files changed, 36 insertions(+), 52 deletions(-)

diff --git a/openbox/menuframe.c b/openbox/menuframe.c
index dd37212..61632f9 100644
--- a/openbox/menuframe.c
+++ b/openbox/menuframe.c
@@ -232,41 +232,13 @@ void menu_frame_move(ObMenuFrame *self, gint x, gint y)
     XMoveWindow(obt_display, self->window, self->area.x, self->area.y);
 }
 
-static void calc_position(ObMenuFrame *self, GravityPoint *position,
-                          gint *x, gint *y, gint monitor)
-{
-    const Rect *area = screen_physical_area_monitor(monitor);
-
-    if (position->x.center)
-        *x = area->width / 2 - self->area.width / 2;
-    else {
-        *x = position->x.pos;
-        if (position->x.denom)
-            *x = (*x * area->width) / position->x.denom;
-        if (position->x.opposite)
-            *x = area->width - self->area.width - *x;
-    }
-
-    if (position->y.center)
-        *y = area->height / 2 - self->area.height / 2;
-    else {
-        *y = position->y.pos;
-        if (position->y.denom)
-            *y = (*y * area->height) / position->y.denom;
-        if (position->y.opposite)
-            *y = area->height - self->area.height - *y;
-    }
-
-    *x += area->x;
-    *y += area->y;
-}
-
 static void menu_frame_place_topmenu(ObMenuFrame *self, GravityPoint *pos,
                                      gint *x, gint *y, gint monitor)
 {
     gint dx, dy;
 
-    calc_position(self, pos, x, y, monitor);
+    screen_apply_gravity_point(pos, x, y, screen_physical_area_monitor(monitor),
+                               self->area.width, self->area.height);
 
     if (config_menu_middle) {
 
diff --git a/openbox/place.c b/openbox/place.c
index 1dc16c4..8136635 100644
--- a/openbox/place.c
+++ b/openbox/place.c
@@ -295,28 +295,8 @@ static gboolean place_per_app_setting_position(ObClient *client, Rect *screen,
 
     ob_debug("placing by per-app settings");
 
-    if (settings->position.x.center)
-        *x = screen->width / 2 - client->area.width / 2;
-    else {
-        *x = settings->position.x.pos;
-        if (settings->position.x.denom)
-            *x = (*x * screen->width) / settings->position.x.denom;
-        if (settings->position.x.opposite)
-            *x = screen->width - frame_size.width - *x;
-    }
-
-    if (settings->position.y.center)
-        *y = screen->height / 2 - client->area.height / 2;
-    else {
-        *y = settings->position.y.pos;
-        if (settings->position.y.denom)
-            *y = (*y * screen->height) / settings->position.y.denom;
-        if (settings->position.y.opposite)
-            *y = screen->height - frame_size.height - *y;
-    }
-
-    *x += screen->x;
-    *y += screen->y;
+    screen_apply_gravity_point(&settings->position, x, y,
+                               screen, frame_size.width, frame_size.height);
 
     return TRUE;
 }
diff --git a/openbox/screen.c b/openbox/screen.c
index c5b97fb..7bd9a61 100644
--- a/openbox/screen.c
+++ b/openbox/screen.c
@@ -1953,3 +1953,30 @@ gboolean screen_compare_desktops(guint a, guint b)
         b = screen_desktop;
     return a == b;
 }
+
+void screen_apply_gravity_point(GravityPoint *position, gint *x, gint *y,
+                                const Rect *area, gint width, gint height)
+{
+    if (position->x.center)
+        *x = area->width / 2 - width / 2;
+    else {
+        *x = position->x.pos;
+        if (position->x.denom)
+            *x = (*x * area->width) / position->x.denom;
+        if (position->x.opposite)
+            *x = area->width - width - *x;
+    }
+
+    if (position->y.center)
+        *y = area->height / 2 - height / 2;
+    else {
+        *y = position->y.pos;
+        if (position->y.denom)
+            *y = (*y * area->height) / position->y.denom;
+        if (position->y.opposite)
+            *y = area->height - height - *y;
+    }
+
+    *x += area->x;
+    *y += area->y;
+}
diff --git a/openbox/screen.h b/openbox/screen.h
index 673a994..bfa084b 100644
--- a/openbox/screen.h
+++ b/openbox/screen.h
@@ -182,4 +182,9 @@ guint screen_monitor_pointer(void);
 */
 gboolean screen_compare_desktops(guint a, guint b);
 
+/*! Resolve a gravity point into absolute coordinates.
+ * width and height are the size of the object being placed, used for
+ * aligning to right/bottom edges of the area. */
+void screen_apply_gravity_point(GravityPoint *position, gint *x, gint *y,
+                                const Rect *area, gint width, gint height);
 #endif
-- 
1.9.1