On Sat, Mar 06, 2021 at 04:01:10PM +0100 I heard the voice of Rhialto, and lo! it spake thus: > On Fri 05 Mar 2021 at 17:51:39 -0600, Matthew D. Fuller wrote: > > > - The NULL check removals in the ring manglement funcs. Sounds > > reasonably in general, but wanted to be sure you checked it in edge > > cases; e.g., if there's nothing in the ring or the like, might we > > have NULL's snuck in somehow? > > I did check edge cases with removing all windows from the ring one by > one and adding some back. It turned out that the Occupy Window is also > on the ring so it isn't really empty yet when you would think it is.
Hey, guess what; when you delete a window (say, exit an xterm), we call the Unlink. If the window isn't on the ring (say, you have no WindowRing stuff enabled in your config, and you never f.ring'd it), guess what's in next/prev? Turns out _I_ know the answer to that one now; ask me how I learned it... :p -- Matthew Fuller (MF4839) | fulle...@over-yonder.net Systems/Network Administrator | http://www.over-yonder.net/~fullermd/ On the Internet, nobody can hear you scream.