In message <[EMAIL PROTECTED]> on Sun, 02 Dec 2007 15:39:38 -0800, "Martin 
Blais" <[EMAIL PROTECTED]> said:

blais> Allright, Rhialto, I looked into it, I'm pretty convinced that
blais> these comments out bits were just for debugging stuff.
blais> Sorry about that.
blais> Here is a patch that brings back these, and the SaveWorkspaceFeature
blais> still works fine on my box.

Added, committed, pushed.

I'm wondering if you missed something, though (see the 2 lines
starting with "!!"):

  > @@ -639,7 +635,7 @@ void GotoWorkSpace (virtualScreen *vs, W
!!>  /*     /\* keep track of the order of the workspaces across restarts *\/ */
!!>  /*     CtwmSetVScreenMap(dpy, Scr->Root, Scr->vScreenList); */
  >  
  > -/*     XSync (dpy, 0); */
  > +    XSync (dpy, 0);
  >      if (Scr->ClickToFocus || Scr->SloppyFocus) set_last_window (newws);
  >      MaybeAnimate = True;
  >  }

Are those lines supposed to stay?

(yeah, I know, I'm picking nits, but if can keep the code a little bit
cleaner, it's probably gonna be a win in the long run)

Cheers,
Richard

-----
Please consider sponsoring my work on free software.
See http://www.free.lp.se/sponsoring.html for details.

-- 
Richard Levitte                         [EMAIL PROTECTED]
                                        http://richard.levitte.org/

"When I became a man I put away childish things, including
 the fear of childishness and the desire to be very grown up."
                                                -- C.S. Lewis

Reply via email to