Remove the default address parameter from QDBusServer's constructor.
authorRaphael Kubo da Costa <rakuco@FreeBSD.org>
Tue, 17 Jan 2012 03:10:46 +0000 (01:10 -0200)
committerQt by Nokia <qt-info@nokia.com>
Tue, 17 Jan 2012 16:41:14 +0000 (17:41 +0100)
commit3de1e6f26b692a8261b40decc2d81286b01c1461
tree3be14a17fa171cb8bacd4844dabd27e47590af19
parent314da0ae017d664b3f05e6474f17c6610fb0257f
Remove the default address parameter from QDBusServer's constructor.

Commit 5be6cf0a6e306ed3a51ed5ba89317b1317544eea introduced an implicit
cast from const char* to QString in QDBusServer's constructor, which
breaks the compilation of applications which use QtDBus when
QT_NO_CAST_FROM_ASCII is defined and clang is used.

Fix it by splitting the current constructor with the broken default
argument into one which takes a non-default QString and one which only
takes a QObject* parent and calls the other with the current default
argument.

It would have been better not to have mostly duplicate code in both
constructors, but QDBusConnectionPrivate is also used in other places.

Task-number: QTBUG-23398
Change-Id: Ia001d63878e7ff720c6630a3372adc571124448d
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
src/dbus/qdbusserver.cpp
src/dbus/qdbusserver.h