fixed for the coding rules

Change-Id: I15d4103834393b4ed702bc9972a0b2ffb6350d20

diff --git a/heremaps-uc-launcher/src/heremaps-uc-launcher.h b/heremaps-uc-launcher/src/heremaps-uc-launcher.h
index fbbf2a2..9749068 100644 (file)
--- a/heremaps-uc-launcher/src/heremaps-uc-launcher.h
+++ b/heremaps-uc-launcher/src/heremaps-uc-launcher.h
@@ -52,10 +52,10 @@ extern "C" {
 #define LOG_TAG TAG_UC_LAUNCHER
 #endif
 
-#define LS_LOGD(fmt,args...)  LOGD(fmt, ##args)
-#define LS_LOGW(fmt,args...)  LOGW(fmt, ##args)
-#define LS_LOGI(fmt,args...)  LOGI(fmt, ##args)
-#define LS_LOGE(fmt,args...)  LOGE(fmt, ##args)
+#define LS_LOGD(fmt, args...)  LOGD(fmt, ##args)
+#define LS_LOGW(fmt, args...)  LOGW(fmt, ##args)
+#define LS_LOGI(fmt, args...)  LOGI(fmt, ##args)
+#define LS_LOGE(fmt, args...)  LOGE(fmt, ##args)
 
 #elif UC_LAUNCHER_DLOG_RELEASE      /* if release mode */
 
@@ -64,10 +64,10 @@ extern "C" {
 #define LOG_TAG TAG_UC_LAUNCHER
 #endif
 
-#define LS_LOGD(fmt,args...)  LOGD(fmt, ##args)
-#define LS_LOGW(fmt,args...)  LOGW(fmt, ##args)
-#define LS_LOGI(fmt,args...)  LOGI(fmt, ##args)
-#define LS_LOGE(fmt,args...)  LOGE(fmt, ##args)
+#define LS_LOGD(fmt, args...)  LOGD(fmt, ##args)
+#define LS_LOGW(fmt, args...)  LOGW(fmt, ##args)
+#define LS_LOGI(fmt, args...)  LOGI(fmt, ##args)
+#define LS_LOGE(fmt, args...)  LOGE(fmt, ##args)
 #else                       /* if do not use dlog */
 #define LS_LOGD(...)  g_debug(__VA_ARGS__)
 #define LS_LOGW(...)  g_warning(__VA_ARGS__)

diff --git a/heremaps-uc/src/heremaps-uc-common.h b/heremaps-uc/src/heremaps-uc-common.h
index 82349a2..b948a1c 100644 (file)
--- a/heremaps-uc/src/heremaps-uc-common.h
+++ b/heremaps-uc/src/heremaps-uc-common.h
@@ -59,10 +59,10 @@ extern "C" {
 #define LOG_TAG TAG_HEREMAPS_UC
 #endif
 
-#define LS_LOGD(fmt,args...)  LOGD(fmt, ##args)
-#define LS_LOGW(fmt,args...)  LOGW(fmt, ##args)
-#define LS_LOGI(fmt,args...)  LOGI(fmt, ##args)
-#define LS_LOGE(fmt,args...)  LOGE(fmt, ##args)
+#define LS_LOGD(fmt, args...)  LOGD(fmt, ##args)
+#define LS_LOGW(fmt, args...)  LOGW(fmt, ##args)
+#define LS_LOGI(fmt, args...)  LOGI(fmt, ##args)
+#define LS_LOGE(fmt, args...)  LOGE(fmt, ##args)
 
 #elif HEREMAPS_UC_DLOG_RELEASE      /* if release mode */
 
@@ -71,10 +71,10 @@ extern "C" {
 #define LOG_TAG TAG_HEREMAPS_UC
 #endif
 
-#define LS_LOGD(fmt,args...)  LOGD(fmt, ##args)
-#define LS_LOGW(fmt,args...)  LOGW(fmt, ##args)
-#define LS_LOGI(fmt,args...)  LOGI(fmt, ##args)
-#define LS_LOGE(fmt,args...)  LOGE(fmt, ##args)
+#define LS_LOGD(fmt, args...)  LOGD(fmt, ##args)
+#define LS_LOGW(fmt, args...)  LOGW(fmt, ##args)
+#define LS_LOGI(fmt, args...)  LOGI(fmt, ##args)
+#define LS_LOGE(fmt, args...)  LOGE(fmt, ##args)
 #else                       /* if do not use dlog */
 #define LS_LOGD(...)  g_debug(__VA_ARGS__)
 #define LS_LOGW(...)  g_warning(__VA_ARGS__)

diff --git a/inc/heremaps-uc-dbus.h b/inc/heremaps-uc-dbus.h
index 229845f..1626dc9 100644 (file)
--- a/inc/heremaps-uc-dbus.h
+++ b/inc/heremaps-uc-dbus.h
@@ -33,11 +33,11 @@ extern "C" {
 #endif
 #define LOG_TAG "HERE_PLUGIN"
 
-#define MAPS_LOGD(fmt,args...)  LOGD(fmt, ##args)
-#define MAPS_LOGW(fmt,args...)  LOGW(fmt, ##args)
-#define MAPS_LOGI(fmt,args...)  LOGI(fmt, ##args)
-#define MAPS_LOGE(fmt,args...)  LOGE(fmt, ##args)
-#define MAPS_SECLOG(fmt,args...)  SECURE_LOGD(fmt, ##args)
+#define MAPS_LOGD(fmt, args...)  LOGD(fmt, ##args)
+#define MAPS_LOGW(fmt, args...)  LOGW(fmt, ##args)
+#define MAPS_LOGI(fmt, args...)  LOGI(fmt, ##args)
+#define MAPS_LOGE(fmt, args...)  LOGE(fmt, ##args)
+#define MAPS_SECLOG(fmt, args...)  SECURE_LOGD(fmt, ##args)
 
 #define MAPS_FUNC_ENTER     MAPS_LOGD("(%s) ENTER", __FUNCTION__);
 #define MAPS_FUNC_EXIT      MAPS_LOGD("(%s) EXIT", __FUNCTION__);

diff --git a/src/here_manager.cpp b/src/here_manager.cpp
index 0d9f592..632d19b 100644 (file)
--- a/src/here_manager.cpp
+++ b/src/here_manager.cpp
@@ -506,12 +506,12 @@ here_error_e HereManager::CheckAgreement()
 
                pid_t nProcessId = getpid();
                ret = app_manager_get_app_id(nProcessId, &strAppId);
-               if (ret != APP_MANAGER_ERROR_NONE)
+               if (ret != APP_MANAGER_ERROR_NONE) {
                        MAPS_LOGI("Get app_id [%ld]. nRet[%d]", nProcessId, ret);
-               else if (!strncmp(strAppId, UTC_APP, strlen(UTC_APP)) ||
+               } else if (!strncmp(strAppId, UTC_APP, strlen(UTC_APP)) ||
                                !strncmp(strAppId, ITC_APP, strlen(ITC_APP)) ||
                                !strncmp(strAppId, UTC_TPK_APP, strlen(UTC_TPK_APP)) ||
-                               !strncmp(strAppId, ITC_TPK_APP, strlen(ITC_TPK_APP))) { 
+                               !strncmp(strAppId, ITC_TPK_APP, strlen(ITC_TPK_APP))) {
                        MAPS_LOGD("Requested by tct");
                        error = HERE_ERROR_NONE;
                }
@@ -522,8 +522,9 @@ here_error_e HereManager::CheckAgreement()
                ret = heremaps_uc_dbus_launch_receiver();
                if (ret != HEREMAPS_UC_DBUS_ERROR_NONE)
                        MAPS_LOGD("heremaps_uc_dbus_launch_receiver fail");
-       } else
+       } else {
                MAPS_LOGD("Vconf value of HereMaps is true");
+       }
 
        if (strAppId != NULL)
                g_free(strAppId);