[Kde-bindings] Wrapper for KWin class ?

[Richard Dale]

On Wednesday 25 January 2006 20:42, Han Holl wrote:
> On Wednesday 25 January 2006 21:25, Richard Dale wrote:
> > The kwin.h header has #ifdef Q_OS_UNIX's round the KWin class, and
> > kalyptus currently skips anything OS specific like that. I did try
> > removing the skipping code, and the Smoke library built fine, so maybe
> > that could be fixed as KDE will only build on Unix-like systems at the
> > moment.
>
> That would be nice.
>
> Most things you can do though KWin can be accomplished via Window-Specific
> Settings in KControl, but it's nicer to do this from the program itself.
> Could I try this for myself ? I do still have the entire kdebindings
> directory tree after the build.
> I see a relevant entry in the kalyptus changelog, but I don't see an
> obvious patch.
I haven't checked anything in. It really needs to do a config test, and skip 
Q_OS_ #ifdefs which aren't for the current OS. Here are the two lines to 
comment out in kalyptus anyway so you can try it out:

Index: kalyptus
===================================================================
--- kalyptus    (revision 493358)
+++ kalyptus    (working copy)
@@ -581,8 +581,8 @@
                                     ($p =~ m/^#\s*if\s+defined\(_WS_MAC_/ and 
$qt_embedded) or
                                     ($p =~ 
m/^#\s*if\s+defined\(Q_INCOMPATIBLE_3_0_ADDONS/ and $qt_embedded) or
                                     $p =~ m/^#\s*ifndef\s+QT_NO_STL/ or
-                                    $p =~ m/^#\s*if\s+defined\s*\(Q_OS_/ or
-                                    $p =~ m/^#\s*ifdef\s*Q_OS_/ or
+#                                   $p =~ m/^#\s*if\s+defined\s*\(Q_OS_/ or
+#                                   $p =~ m/^#\s*ifdef\s*Q_OS_/ or
                                     $p =~ m/^#\s*if\s+defined\(Q_CC_/ or
                                     $p =~ 
m/^#\s*if\s+defined\(QT_THREAD_SUPPORT/ or
                                     $p =~ m/^#\s*else/ or


-- Richard