Johann Haarhoff wrote:
>> + tmp = wwin;
>> + /* TODO: Is the focused window always the last in the list? */
>> + while (tmp->prev) {
>
> Perhaps instead of iterating the window list yourself, you could use
> makeWindowListArray in switchpanel.c which will populate an array of
> windows for you without worrying about the question in your TODO?
I can remove the TODO: now (it should really be TOLEARN instead :-)
The focused window is always the last in the list. I just found two
or three places in actions.c and sessions.c where this is implicitly
assumed (and the code works). Besides, my tests when I was writing
Maximus also showed that.
> BTW, has anybody tested this patch, or my Left/Right Maximize patch on a
> multi-head setup. Do these patches behave as expected on a Xinerama setup?
>
> I don't have a multi-head setup myself, is it possible to duplicate with
> Xnest? How does one go about testing this?
Yeah, I would like to know too. I don't have a xinerama setup.
--
To unsubscribe, send mail to [email protected].
