New rootless mode
Hi, guys. I'm writing code related mwextwm. Temporally I added -internalwm option. It detects another window manager, and automatically it switches on/off built-in window manager. There are duplicated code for -rootless and -multiwindow and -mwextwm. This will merge those three mode. I have to clean-up, debug and test more. But I want to know direction of this patch before. How do you think? 1. Add -internalwm option? (more better naming?) 2. Replace -rootless and -multiwindow with this. or something else. -- Kensuke Matsuzaki mailto:[EMAIL PROTECTED] http://peppermint.jp internalwm-20041030-0050.diff Description: Binary data
Re: New rootless mode
Kensuke Matsuzaki wrote: > Hi, guys. > > I'm writing code related mwextwm. Temporally I added -internalwm > option. It detects another window manager, and automatically it > switches on/off built-in window manager. There are duplicated code for > -rootless and -multiwindow and -mwextwm. This will merge those three > mode. > > I have to clean-up, debug and test more. But I want to know direction > of this patch before. How do you think? > > 1. Add -internalwm option? (more better naming?) -internalwm is actually -multiwindow? > 2. Replace -rootless and -multiwindow with this. > or something else. I'd keep the options (at least their names) for backward compatibility. BTW: Torrey wanted to change a rootless definition in fb.h https://freedesktop.org/bugzilla/show_bug.cgi?id=1737 Can you please take a look? I don't think the change will break anything but I've not the complete insight on the code. bye ago NP: Neikka RPM - Bound with sympathy (Terrorfakt mix) -- [EMAIL PROTECTED] http://www.gotti.org ICQ: 126018723
Re: New rootless mode
> -internalwm is actually -multiwindow? No. -multiwindow use hw/xwin/winmultiwindow*.c to handle widow, but -internalwm use miext/rootless/. And -internalwm uses built-in wm in winmultiwindowwm.c too if another window manager is running. > I'd keep the options (at least their names) for backward compatibility. I see. > BTW: Torrey wanted to change a rootless definition in fb.h > https://freedesktop.org/bugzilla/show_bug.cgi?id=1737 > > Can you please take a look? I don't think the change will break anything > but I've not the complete insight on the code. It seems no problem. I remember that I added defined(__CYGWIN__) to use miext/rootless. http://freedesktop.org/cgi-bin/viewcvs.cgi/xc/programs/Xserver/fb/fb.h?r1=1.1.4.1&root=xorg&sortby=date&r2=1.1.4.1.2.1&only_with_tag=CYGWIN Thank you. -- Kensuke Matsuzaki mailto:[EMAIL PROTECTED] http://peppermint.jp
Re: New rootless mode
Kensuke Matsuzaki wrote: > > -internalwm is actually -multiwindow? > > No. -multiwindow use hw/xwin/winmultiwindow*.c to handle widow, but > -internalwm use miext/rootless/. And -internalwm uses built-in wm in > winmultiwindowwm.c too if another window manager is running. oh. This makes it clear. Just go ahead. This sounds very useful. > > BTW: Torrey wanted to change a rootless definition in fb.h > > https://freedesktop.org/bugzilla/show_bug.cgi?id=1737 > > > > Can you please take a look? I don't think the change will break anything > > but I've not the complete insight on the code. > > It seems no problem. I remember that I added defined(__CYGWIN__) to use > miext/rootless. > http://freedesktop.org/cgi-bin/viewcvs.cgi/xc/programs/Xserver/fb/fb.h?r1=1.1.4.1&root=xorg&sortby=date&r2=1.1.4.1.2.1&only_with_tag=CYGWIN I'll tell Torrey to go on. bye ago NP: Interlace - Elohim -- [EMAIL PROTECTED] http://www.gotti.org ICQ: 126018723