[FMRadio] update logs
authorLukasz Bardeli <l.bardeli@samsung.com>
Thu, 26 Nov 2015 14:06:48 +0000 (15:06 +0100)
committerLukasz Bardeli <l.bardeli@samsung.com>
Thu, 26 Nov 2015 14:06:48 +0000 (15:06 +0100)
Verification] Code compiles without errors

Change-Id: I92e1665e3b510547be95b756a089ff352478ed59
Signed-off-by: Lukasz Bardeli <l.bardeli@samsung.com>
src/radio/radio_instance.cc
src/radio/radio_manager.cc

index 4773f4ba012a7389854cbb55a33de99ba5b0d21c..6e218747d57027c35ae069b5c75d6f0827434ae3 100644 (file)
@@ -123,8 +123,7 @@ void RadioInstance::Start(const picojson::value& args, picojson::object& out) {
   if (result) {
     ReportSuccess(out);
   } else {
-    LoggerE("Failed");
-    ReportError(result, &out);
+    LogAndReportError(result, &out);
   }
 }
 
@@ -136,8 +135,7 @@ void RadioInstance::Stop(const picojson::value& args, picojson::object& out) {
   if (result) {
     ReportSuccess(out);
   } else {
-    LoggerE("Failed");
-    ReportError(result, &out);
+    LogAndReportError(result, &out);
   }
 }
 
@@ -164,8 +162,7 @@ void RadioInstance::SetFMRadioInterruptedListener(const picojson::value& args,
   if (result) {
     ReportSuccess(out);
   } else {
-    LoggerE("Failed");
-    ReportError(result, &out);
+    LogAndReportError(result, &out);
   }
 }
 
@@ -178,8 +175,7 @@ void RadioInstance::UnsetFMRadioInterruptedListener(const picojson::value& args,
   if (result) {
     ReportSuccess(out);
   } else {
-    LoggerE("Failed");
-    ReportError(result, &out);
+    LogAndReportError(result, &out);
   }
 }
 
@@ -192,8 +188,7 @@ void RadioInstance::SetAntennaChangeListener(const picojson::value& args,
   if (result) {
     ReportSuccess(out);
   } else {
-    LoggerE("Failed");
-    ReportError(result, &out);
+    LogAndReportError(result, &out);
   }
 }
 
@@ -206,8 +201,7 @@ void RadioInstance::UnsetAntennaChangeListener(const picojson::value& args,
   if (result) {
     ReportSuccess(out);
   } else {
-    LoggerE("Failed");
-    ReportError(result, &out);
+    LogAndReportError(result, &out);
   }
 }
 
index 81bb0de276f64870001b40c450a3c83b4fb48717..84931ed2a64bba9e8b1184cbc4abd3980e0f7036 100755 (executable)
@@ -31,6 +31,7 @@
 #include "radio/radio_instance.h"
 
 using namespace common;
+using namespace common::tools;
 using namespace std;
 
 namespace extension {
@@ -60,16 +61,16 @@ PlatformResult GetPlatformResult(const std::string& str, int err) {
 
   switch (err) {
     case RADIO_ERROR_INVALID_PARAMETER:
-      return PlatformResult(ErrorCode::INVALID_VALUES_ERR, message);
+      return LogAndCreateResult(ErrorCode::INVALID_VALUES_ERR, message);
 
     case RADIO_ERROR_INVALID_STATE:
-      return PlatformResult(ErrorCode::INVALID_STATE_ERR, message);
+      return LogAndCreateResult(ErrorCode::INVALID_STATE_ERR, message);
 
     case RADIO_ERROR_NOT_SUPPORTED:
-      return PlatformResult(ErrorCode::SERVICE_NOT_AVAILABLE_ERR, message);
+      return LogAndCreateResult(ErrorCode::SERVICE_NOT_AVAILABLE_ERR, message);
 
     default:
-      return PlatformResult(ErrorCode::UNKNOWN_ERR, message);
+      return LogAndCreateResult(ErrorCode::UNKNOWN_ERR, message);
   }
 }
 
@@ -368,7 +369,7 @@ PlatformResult FMRadioManager::Start(double frequency) {
   }
 
   if (RADIO_STATE_READY != state && RADIO_STATE_PLAYING != state) {
-    return PlatformResult(ErrorCode::INVALID_STATE_ERR, "Invalid radio state.");
+    return LogAndCreateResult(ErrorCode::INVALID_STATE_ERR, "Invalid radio state.");
   }
 
   PlatformResult result = SetFrequency(frequency);
@@ -396,7 +397,7 @@ PlatformResult FMRadioManager::Stop() {
   }
 
   if (RADIO_STATE_PLAYING != state) {
-    return PlatformResult(ErrorCode::INVALID_STATE_ERR, "Invalid radio state.");
+    return LogAndCreateResult(ErrorCode::INVALID_STATE_ERR, "Invalid radio state.");
   }
 
   return CheckError("radio_stop", radio_stop(radio_instance_));
@@ -535,7 +536,7 @@ void FMRadioManager::PostResultFailure(double callbackId, const PlatformResult&
   picojson::value event{picojson::object()};
   auto& obj = event.get<picojson::object>();
 
-  tools::ReportError(result, &obj);
+  LogAndReportError(result, &obj);
   AddCallbackID(callbackId, &obj);
 
   PostMessage(event.serialize());