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

List:       kde-devel
Subject:    Re: KDE, Qt3 and Amarok
From:       "Bart Van Assche" <bart.vanassche () gmail ! com>
Date:       2008-04-22 19:12:43
Message-ID: e2e108260804221212k39c0088aob58f8fa7e98da46b () mail ! gmail ! com
[Download RAW message or body]

On Tue, Apr 22, 2008 at 8:34 AM, Thiago Macieira <thiago@kde.org> wrote:
> Bart Van Assche wrote:
>  >Do you know where I can find more information about the portability
>  >requirements for Qt 3 patches for KDE ? Is it allowed to use gcc
>  >extensions ? And are other threading libraries than POSIX threads
>  >supported ?
>
>  Well, since a patch is only a suggestion, you can do whatever you want
>  with it. If it doesn't compile for other platforms, they can just not
>  apply your patch. Or they can fix it and send it back to you :-)

Something like this ?

diff -upr orig/qt-x11-free-3.3.7/src/tools/qshared.h
qt-x11-free-3.3.7/src/tools/qshared.h
--- orig/qt-x11-free-3.3.7/src/tools/qshared.h  2006-10-19
16:25:03.000000000 +0200
+++ qt-x11-free-3.3.7/src/tools/qshared.h       2008-04-04
14:01:26.000000000 +0200
@@ -46,8 +46,8 @@
 struct Q_EXPORT QShared
 {
    QShared() : count( 1 ) { }
-    void ref()         { count++; }
-    bool deref()       { return !--count; }
+    void ref()         { __sync_add_and_fetch(&count, 1); }
+    bool deref()       { return ! __sync_sub_and_fetch(&count, 1); }
    uint count;
 };

Bart.
 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread]