[patch] KUniqueApplication: dbus service cleanup for Windows

[Rafael Fernández López]

Hi,

> I'm not in a position to comment on your approach, but to avoid the code
> looking at first sight like '==' has accidentally been written as '=', it
> would read better as:
>
> +  if (QDBusConnection::sessionBus().isConnected()
> +    && (dbusService = QDBusConnection::sessionBus().interface()) != 0)

Nope, it is fine. It assigns to dbusService the address of the interface, and 
later checks if it was different to NULL. It is fine.

In other case (using ==), you would be generating a boolean expression. Is a 
different thing.

This is the same as writing:

if (QDBusConnection::sessionBus().isConnected()) {
    dbusService = QDBusConnection::sessionBus().interface();
    if (dbusService) {
        ...
    }
}


Bye,
Rafael Fernández López

GPG Fingerprint: B9F4 4730 43F8 FFDD CC5E BA8E 724E 406E 3F01 D070
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080313/acd57ca0/attachment.sig>