Replace int with pid_t where applicable

Change-Id: Ic388b17e4b37072eda5861d39b92345b8553d55b

diff --git a/inc/application_manager.h b/inc/application_manager.h
index 0bb6fea..1effd84 100644 (file)
--- a/inc/application_manager.h
+++ b/inc/application_manager.h
@@ -27,11 +27,11 @@ extern "C" {
 
 class IApplicationManager {
 public:
-       virtual bool is_application_running(int pid) = 0;
+       virtual bool is_application_running(pid_t pid) = 0;
        virtual bool is_application_running(const std::string& appid) = 0;
        virtual bool bring_app_to_foreground(const std::string& appid) = 0;
        virtual bool launch_app_async(const std::string& appid, bool background) = 0;
-       virtual boost::optional<std::string> get_appid_by_pid(int pid) = 0;
+       virtual boost::optional<std::string> get_appid_by_pid(pid_t pid) = 0;
 };
 
 #ifdef __cplusplus

diff --git a/inc/application_manager_aul.h b/inc/application_manager_aul.h
index 0de9479..4b46f59 100644 (file)
--- a/inc/application_manager_aul.h
+++ b/inc/application_manager_aul.h
@@ -29,11 +29,11 @@ public:
        CApplicationManagerAul();
        ~CApplicationManagerAul();
 
-       virtual bool is_application_running(int pid) override;
+       virtual bool is_application_running(pid_t pid) override;
        virtual bool is_application_running(const std::string& appid) override;
        virtual bool bring_app_to_foreground(const std::string& appid) override;
        virtual bool launch_app_async(const std::string& appid, bool background) override;
-       virtual boost::optional<std::string> get_appid_by_pid(int pid) override;
+       virtual boost::optional<std::string> get_appid_by_pid(pid_t pid) override;
 };
 
 #ifdef __cplusplus

diff --git a/src/application_manager_aul.cpp b/src/application_manager_aul.cpp
index 9418b96..41ae5f2 100644 (file)
--- a/src/application_manager_aul.cpp
+++ b/src/application_manager_aul.cpp
@@ -28,7 +28,7 @@ CApplicationManagerAul::~CApplicationManagerAul()
 {
 }
 
-bool CApplicationManagerAul::is_application_running(int pid)
+bool CApplicationManagerAul::is_application_running(pid_t pid)
 {
        int status = aul_app_get_status_bypid(pid);
        if (0 > status) {
@@ -98,7 +98,7 @@ bool CApplicationManagerAul::launch_app_async(const std::string& appid, bool bac
        return ret;
 }
 
-boost::optional<std::string> CApplicationManagerAul::get_appid_by_pid(int pid)
+boost::optional<std::string> CApplicationManagerAul::get_appid_by_pid(pid_t pid)
 {
        boost::optional<std::string> ret;
 

diff --git a/tests/utc/client-manager/test_client_manager.cpp b/tests/utc/client-manager/test_client_manager.cpp
index d23428d..da5cb94 100644 (file)
--- a/tests/utc/client-manager/test_client_manager.cpp
+++ b/tests/utc/client-manager/test_client_manager.cpp
@@ -26,11 +26,11 @@ public:
        CDummyApplicationManager() {};
        virtual ~CDummyApplicationManager() {};
 
-       bool is_application_running(int pid) { return true; }
+       bool is_application_running(pid_t pid) { return true; }
        bool is_application_running(const std::string& appid) { return true; }
        bool bring_app_to_foreground(const std::string& appid) { return true; }
        bool launch_app_async(const std::string& appid, bool background) { return true; }
-       boost::optional<std::string> get_appid_by_pid(int pid) { return boost::optional<std::string>{}; }
+       boost::optional<std::string> get_appid_by_pid(pid_t pid) { return boost::optional<std::string>{}; }
 };
 
 class StorageWithNoClient : public testing::Test

diff --git a/tests/utc/service-main/test_service_main.cpp b/tests/utc/service-main/test_service_main.cpp
index b7313c5..b14cf72 100644 (file)
--- a/tests/utc/service-main/test_service_main.cpp
+++ b/tests/utc/service-main/test_service_main.cpp
@@ -28,7 +28,7 @@ public:
        CDummyApplicationManager() {};
        ~CDummyApplicationManager() {};
 
-       bool is_application_running(int pid) override { return false; }
+       bool is_application_running(pid_t pid) override { return false; }
        bool is_application_running(const std::string& appid) override { return false; }
        bool bring_app_to_foreground(const std::string& appid) override
        {
@@ -41,7 +41,7 @@ public:
                launched_option_background = background;
                return true;
        }
-       boost::optional<std::string> get_appid_by_pid(int pid) override { return boost::optional<std::string>{}; }
+       boost::optional<std::string> get_appid_by_pid(pid_t pid) override { return boost::optional<std::string>{}; }
 public:
        boost::optional<std::string> launched_appid;
        boost::optional<bool> launched_option_background;