Re: New rootless mode

2004-10-30 Thread Alexander Gottwald
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

2004-10-30 Thread Kensuke Matsuzaki
 -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.1root=xorgsortby=dater2=1.1.4.1.2.1only_with_tag=CYGWIN

Thank you.
-- 
Kensuke Matsuzaki
mailto:[EMAIL PROTECTED]
http://peppermint.jp


Re: New rootless mode

2004-10-30 Thread Alexander Gottwald
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.1root=xorgsortby=dater2=1.1.4.1.2.1only_with_tag=CYGWIN

I'll tell Torrey to go on.

bye
ago
NP: Interlace - Elohim
-- 
 [EMAIL PROTECTED]
 http://www.gotti.org   ICQ: 126018723