fixed the warning message when enabling -Wall option

This patch fixed the below warning messages.
- warning: comparison between signed and unsigned integer expressions
- warning: no return statement in function returning non-void
- warning: statement has no effect
- warning: unused variable
- warning: when initialized here [-Wreorder]
- warning: control reaches end of non-void function [-Wreturn-type]
- warning: ‘VehicleProperty::ButtonEvent’ is deprecated: Deprecated in
           0.14.  Use ButtonEventW3C [-Wdeprecated-declarations]
- warning: ignoring return value of 'int dup(int)', declared with
           attribute warn_unused_result [-Wunused-result]
- warning: suggest parentheses around '&&' within '||' [-Wparentheses]
- warning: 'connection' may be used uninitialized in this function
           [-Wmaybe-uninitialized]
- warning: suggest parentheses around assignment used as truth value
- warning: dereferencing type-punned pointer will break strict-aliasing rules

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

1 files changed, 2 insertions, 2 deletions

diff --git a/plugins/common/dbusplugin.cpp b/plugins/common/dbusplugin.cpp
index e56b403b..9e51f94e 100644
--- a/plugins/common/dbusplugin.cpp
+++ b/plugins/common/dbusplugin.cpp
@@ -29,8 +29,8 @@
 std::map<std::string, std::string> DBusSink::dbusConfig;
 
 DBusSink::DBusSink(std::string propertyName, AbstractRoutingEngine* engine, GDBusConnection* connection, std::map<std::string, std::string> config)
-	:AbstractDBusInterface("org.automotive."+propertyName, propertyName, connection),
-	  AbstractSink(engine, dbusConfig)
+	: AbstractSink(engine, dbusConfig),
+	AbstractDBusInterface("org.automotive."+propertyName, propertyName, connection)
 {
 	AbstractDBusInterface::re = engine;