commit f0a4845e7dec3a4c7316311fcf1108148bb29730
from: Anselm R Garbe <anselm@garbe.us>
date: Sat Sep 19 10:52:16 2009 UTC

added TODO to updategeom in order to implement a decent version of it soon

commit - 82ec7a7ed4f3ba376faadfc2aca783b24618575f
commit + f0a4845e7dec3a4c7316311fcf1108148bb29730
blob - 2b4d6b03c137e8e77fa785209ee5999147e4d29b
blob + 73633966d0ba8602baa0e54ae5db10eb67cffb9c
--- dwm.c
+++ dwm.c
@@ -1699,6 +1699,40 @@ updategeom(void) {
 	Client *c;
 	Monitor *newmons = NULL, *m = NULL, *tm;
 
+	/* TODO:
+	 * This function needs to be seriously re-designed:
+	 *
+	 * #ifdef XINERAMA
+	 * 1. Determine number of already existing monitors n
+	 * 2. Determine number of monitors Xinerama reports nn
+	 * 3. if(n <= nn) {
+	 *       if(n < nn) {
+	 *          append nn-n monitors to current struct
+	 *          flag dirty
+	 *       }
+	 *       for(i = 0; i < nn; i++) {
+	 *           if(oldgeom != newgeom) {
+	 *               apply newgeom;
+	 *               flag dirty;
+	 *           }
+	 *       }
+	 *    }
+	 *    else {
+	 *       detach all clients
+	 *       destroy current monitor struct
+	 *       create new monitor struct 
+	 *       attach all clients to first monitor
+	 *       flag dirty;
+	 *    }
+	 *    return dirty flag to caller
+	 *        if dirty is seen by caller:
+	 *           re-arrange bars/pixmaps
+	 *           arrange()
+	 * #else
+	 *    don't share between XINERAMA and non-XINERAMA handling if it gets
+	 *    too ugly
+	 * #endif
+	 */
 #ifdef XINERAMA
 	XineramaScreenInfo *info = NULL;
 	Bool *flags = NULL;