[ctwm] Silly occupy window...

[Matthew D. Fuller]

Today I moved to a new workstation, so I got all the fun of setting up
new monitor and video card and all that fun X stuff.  But post-move, a
minor ctwm issue showed up; calling f.occupy pulled up the occupy
window like normal, except I didn't see any buttons, and the window
was several dozen times as large as my screen in every direction.  Um.
Whoops?

This tracks back to a few variables that are never initialized, unless
some options that I didn't know existed are in the config file.
Presumably it always got zero'd memory on my old system, so I never
noticed (and probably on most other people's systems, 'cuz otherwise
THEY'D have noticed).  I've pushed a fix for it; attached is the
2-line patch in case anybody else runs into it.


-- 
Matthew Fuller (MF4839)   |  fulle...@over-yonder.net
Systems/Network Administrator |  http://www.over-yonder.net/~fullermd/
   On the Internet, nobody can hear you scream.
#
# old_revision [db710969dfa5d7225c4825c75aee971fb4572f8b]
#
# patch workmgr.c
#  from [8ea886eb10be3f1bdf3ae7010f4eb36cbe9382c5]
#to [ea9dbcb2897548e5721e6a6cd2da57223c2dfaad]
#

--- workmgr.c	8ea886eb10be3f1bdf3ae7010f4eb36cbe9382c5
+++ workmgr.c	ea9dbcb2897548e5721e6a6cd2da57223c2dfaad
@@ -143,6 +143,8 @@ void InitWorkSpaceManager (void)
 Scr-workSpaceMgr.occupyWindow-geometry  = NULL;
 Scr-workSpaceMgr.occupyWindow-columns   = 0;
 Scr-workSpaceMgr.occupyWindow-twm_win   = (TwmWindow*) 0;
+Scr-workSpaceMgr.occupyWindow-vspace= Scr-WMgrVertButtonIndent;
+Scr-workSpaceMgr.occupyWindow-hspace= Scr-WMgrHorizButtonIndent;
 
 Scr-workSpaceMgr.curColors.back  = Scr-Black;
 Scr-workSpaceMgr.curColors.fore  = Scr-White;

Re: [ctwm] [rt.lp.se #135] Bugfix for ctwm

[Matthew D. Fuller]

On Tue, Oct 28, 2008 at 09:38:42AM -0500 I heard the voice of
Matthew D. Fuller, and lo! it spake thus:
 
 3) 1000.  This is what ctwm initializes it to if there's no setting
sitting around in a config file.  We can assume that it's infinity.
 
3a) = 1000.  Just in case somebody gets the idea that
$REALLYBIGNUM should be used.
 
 I find myself leaning toward (3).

Well, it's been about 8 months since I posted this.  I choose to take
silence for acceptance of my brilliance, so I've pushed (3a).


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