> So, the right answer is pulling most of the important guts of
> HandleConfigureNotify() into a separate function that HCN() and
> EwmhClientMessage_NET_MOVERESIZE_WINDOW() would both call into.

I've finished this out.  Trunk now has the rewritten
HandleConfigureNotify() path.  It turned out to be simplest to just
[ab]use the XConfigureRequestEvent struct for our internal data
structure too, which makes that wrapping even thinner.

I've adjusted the _NET_MOVERESIZE_WINDOW bits to call into that new
backend func directly as well.  It also allows cleaning up some of the
values that it was putting into the XCRE struct that it didn't
actually need to (but it was difficult to be sure of that before) as
well.  A little wmctrl testing seems to work right (modulo apparent
wmctrl bugs, like the incorrectly claimed handling of -1 values).

Changes up in ewmh_moveresize branch in my fork:
https://github.com/fullermd/ctwm/tree/ewmh_moveresize


-- 
Matthew Fuller     (MF4839)   |  [email protected]
Systems/Network Administrator |  http://www.over-yonder.net/~fullermd/
           On the Internet, nobody can hear you scream.

Reply via email to