diff options
author | JackLivio <jack@livio.io> | 2017-09-29 10:41:50 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-09-29 10:41:50 -0400 |
commit | ce7e33de70e7a07302676aa19aced9a7bb911de0 (patch) | |
tree | 709bf9a8234198be8bd01f7550134c366adb825a | |
parent | 5fd981e582b92930ecb04e25a766eafa022dcceb (diff) | |
parent | e746f8661cb57d105a94663ba8543eaddd381527 (diff) | |
download | sdl_core-ce7e33de70e7a07302676aa19aced9a7bb911de0.tar.gz |
Merge pull request #1797 from smartdevicelink/fix/1789_refix
Fix compilation warning #1789
4 files changed, 11 insertions, 11 deletions
diff --git a/src/appMain/main.cc b/src/appMain/main.cc index 58ab02b123..81482c4335 100644 --- a/src/appMain/main.cc +++ b/src/appMain/main.cc @@ -148,8 +148,7 @@ int32_t main(int32_t argc, char** argv) { // Check if no error values were read from config file if (profile_instance.ErrorOccured()) { - const std::string& error = profile_instance.ErrorDescription(); - LOG4CXX_FATAL(logger_, error); + LOG4CXX_FATAL(logger_, profile_instance.ErrorDescription()); FLUSH_LOGGER(); DEINIT_LOGGER(); exit(EXIT_FAILURE); diff --git a/src/components/application_manager/src/application_manager_impl.cc b/src/components/application_manager/src/application_manager_impl.cc index 411813cc83..1d41388910 100644 --- a/src/components/application_manager/src/application_manager_impl.cc +++ b/src/components/application_manager/src/application_manager_impl.cc @@ -1611,11 +1611,11 @@ void ApplicationManagerImpl::SendMessageToMobile( (*message)[strings::params][strings::function_id].asUInt()); if (function_id == mobile_apis::FunctionID::RegisterAppInterfaceID && (*message)[strings::msg_params][strings::success].asBool()) { - const bool is_for_plugin = plugin_manager_.IsAppForPlugins(app); LOG4CXX_INFO(logger_, - "Registered app " << app->app_id() << " is " - << (is_for_plugin ? "" : "not ") - << "for plugins."); + "Registered app " + << app->app_id() << " is " + << (plugin_manager_.IsAppForPlugins(app) ? "" : "not ") + << "for plugins."); } #endif // SDL_REMOTE_CONTROL } else if (app) { diff --git a/src/components/application_manager/src/policies/policy_handler.cc b/src/components/application_manager/src/policies/policy_handler.cc index ad21f533a1..e6ebaf3b83 100644 --- a/src/components/application_manager/src/policies/policy_handler.cc +++ b/src/components/application_manager/src/policies/policy_handler.cc @@ -808,17 +808,17 @@ bool PolicyHandler::IsAppSuitableForPolicyUpdate( const Applications::value_type value) const { LOG4CXX_AUTO_TRACE(logger_); - const uint32_t app_id = value->app_id(); - if (!value->IsRegistered()) { LOG4CXX_DEBUG(logger_, - "Application " << app_id << " is not marked as registered."); + "Application " << value->app_id() + << " is not marked as registered."); return false; } LOG4CXX_DEBUG(logger_, - "Application " << app_id << " marked as registered." - "Checking its parameters."); + "Application " << value->app_id() + << " marked as registered." + "Checking its parameters."); DeviceParams device_params = GetDeviceParams( value->device(), diff --git a/src/components/policy/policy_regular/src/cache_manager.cc b/src/components/policy/policy_regular/src/cache_manager.cc index b395e4e04c..8b6b9a433e 100644 --- a/src/components/policy/policy_regular/src/cache_manager.cc +++ b/src/components/policy/policy_regular/src/cache_manager.cc @@ -36,6 +36,7 @@ #include <functional> #include <ctime> #include <cmath> +#include <sstream> #include "utils/file_system.h" #include "json/reader.h" |