[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: KWin in the multi-OS API (was: KMainWindow)
From:       Lubos Lunak <l.lunak () suse ! cz>
Date:       2006-11-15 11:08:04
Message-ID: 200611151208.04792.l.lunak () suse ! cz
[Download RAW message or body]

On Wednesday 15 November 2006 00:31, Jarosław Staniek wrote:
> Aaron J. Seigo said the following, On 2006-11-14 17:23:
> > KWin::setState(mainWindow->winId(), Net::FullScreen);
>
> BTW, this reminds me the old TODO: there's no KWin on native mac and
> windows (and even no X11).

 And there doesn't have to be any KWin even on X11 :).

> What about moving stuff like this to more 
> self-documenting names instead of KWin:: and Net:: ?

 There's also a TODO about moving all the functionality from KWin to 
KWinModule (I hate using KWinInternal in KWin the app). I'm fine with calling 
the result KWMModule, KWM (gee :) ) or whatever suitable.

>
> Of course KWin-specific functionality can stay in KWin::, as the
> documentation can warn about its unportability outside of the X11.

 There's not really any KWin-specific functionality in the KWin class. And I 
think it'd be better to simply have every function on every platform, even if 
half of them did nothing on some of them.

 As for the actual classes, all NET* class except for the NET namespace itself 
are X11-specific and they should probably stay that way. They should be only 
used in libs and X11-specific apps, if normal apps need them, that 
functionality should be also made available by the KWin class to have a 
simpler API.

 So the affected classes should be only KWin+KWinModule and NET, but I'm bad 
at making up new names.

-- 
Lubos Lunak
KDE developer
--------------------------------------------------------------
SUSE LINUX, s.r.o.   e-mail: l.lunak@suse.cz , l.lunak@kde.org
Lihovarska 1060/12   tel: +420 284 028 972
190 00 Prague 9      fax: +420 284 028 951
Czech Republic       http//www.suse.cz

[prev in list] [next in list] [prev in thread] [next in thread]