Make some getter methods as immutable methods 52/209252/1
authorJunghoon Park <jh9216.park@samsung.com>
Thu, 4 Jul 2019 02:45:59 +0000 (11:45 +0900)
committerJunghoon Park <jh9216.park@samsung.com>
Thu, 4 Jul 2019 02:49:47 +0000 (11:49 +0900)
- Add 'const' at the end of method definition to make immutable method

Change-Id: I3b05bacf3c386cb0cab4b0726df132632ac53a29
Signed-off-by: Junghoon Park <jh9216.park@samsung.com>
watchface-complication/complication.cc
watchface-complication/complication.h

index 7d997cb..26156b4 100644 (file)
@@ -566,24 +566,24 @@ int Complication::SetCurDataIdx(int cur_data_idx) {
   return impl_->UpdateProviderInfo();
 }
 
-const string Complication::GetCurProviderId() {
+string Complication::GetCurProviderId() const {
   if (!impl_->cur_provider_id_.empty())
     return impl_->cur_provider_id_.c_str();
   return "";
 }
 
-const string Complication::GetProviderId(const Bundle& candidate_data) {
+string Complication::GetProviderId(const Bundle& candidate_data) const {
   return candidate_data.GetString(impl_->provider_id_key_);
 }
 
-int Complication::GetProviderType(const Bundle& candidate_data) {
+int Complication::GetProviderType(const Bundle& candidate_data) const {
   string type = candidate_data.GetString(impl_->provider_type_key_);
   if (type.empty())
     return 0;
   return stoi(type);
 }
 
-int Complication::GetCurType() {
+int Complication::GetCurType() const {
   return static_cast<int>(impl_->cur_type_);
 }
 
index f793bff..ce56a69 100644 (file)
@@ -44,19 +44,24 @@ class EXPORT_API Complication : public IEditable
 
  public:
   class ProviderInfo {
-    std::string provider_id_;
-    int types_;
    public:
     ProviderInfo(std::string provider_id, int types)
       : provider_id_(provider_id), types_(types) {
     }
-    std::string& GetProviderId() {
+
+    const std::string& GetProviderId() const {
       return provider_id_;
     }
-    int GetTypes() {
+
+    int GetTypes() const {
       return types_;
     }
+
+   private:
+    std::string provider_id_;
+    int types_;
   };
+
   int SetHighlight(std::unique_ptr<IEditable::Highlight> highlight) override;
   IEditable::Highlight* GetHighlight() override;
   std::list<std::shared_ptr<tizen_base::Bundle>> const& GetCandidates() const override;
@@ -89,10 +94,10 @@ class EXPORT_API Complication : public IEditable
   const std::string GetSetupAppId() override;
 
   int SendDataUpdateRequest(bool launch_option = true);
-  const std::string GetCurProviderId();
-  const std::string GetProviderId(const tizen_base::Bundle& candidate_data);
-  int GetProviderType(const tizen_base::Bundle& candidate_data);
-  int GetCurType();
+  std::string GetCurProviderId() const;
+  std::string GetProviderId(const tizen_base::Bundle& candidate_data) const;
+  int GetProviderType(const tizen_base::Bundle& candidate_data) const;
+  int GetCurType() const;
   int TouchLaunch(watchface_complication_event_type_e event_type);
   int ApplyAllowedList(
       std::list<std::unique_ptr<ProviderInfo>> allowed_list);