From a7e818f59fed84b8b7db8019a87968cc2eff5ec6 Mon Sep 17 00:00:00 2001
From: Kefu Chai <kchai@redhat.com>
Date: Fri, 21 Jul 2017 21:03:41 +0800
Subject: [PATCH] common/config: change the element type of lists to "const
 char*"

there lists holds constant string literals. and they are not changeable
at runtime. so make std::list<const char*> for smaller memory foot
print.

Signed-off-by: Kefu Chai <kchai@redhat.com>
---
 src/common/options.cc |  2 +-
 src/common/options.h  | 12 ++++++------
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/common/options.cc b/src/common/options.cc
index 4385f83f52a6..1f268c944d37 100644
--- a/src/common/options.cc
+++ b/src/common/options.cc
@@ -5529,7 +5529,7 @@ static std::vector<Option> build_options()
 {
   std::vector<Option> result = global_options;
 
-  auto ingest = [&result](std::vector<Option> &options, const std::string &svc) {
+  auto ingest = [&result](std::vector<Option> &options, const char* svc) {
     for (const auto &o_in : options) {
       Option o(o_in);
       o.add_service(svc);
diff --git a/src/common/options.h b/src/common/options.h
index 9044d975dd33..770fa5314414 100644
--- a/src/common/options.h
+++ b/src/common/options.h
@@ -79,16 +79,16 @@ struct Option {
   // Additionally: "common" for settings that exist in any Ceph code.  Do
   // not use common for settings that are just shared some places: for those
   // places, list them.
-  std::list<std::string> services;
+  std::list<const char*> services;
 
   // Topics like:
   // "service": a catchall for the boring stuff like log/asok paths.
   // "network"
   // "performance": a setting that may need adjustment depending on
   //                environment/workload to get best performance.
-  std::list<std::string> tags;
+  std::list<const char*> tags;
 
-  std::list<std::string> see_also;
+  std::list<const char*> see_also;
 
   value_t min, max;
   std::list<std::string> enum_allowed;
@@ -192,7 +192,7 @@ struct Option {
     return *this;
   }
   Option& add_tag(std::initializer_list<const char*> ts) {
-    tags.insert(tags.end(), ts.begin(), ts.end());
+    tags.insert(tags.end(), ts);
     return *this;
   }
   Option& add_service(const char* service) {
@@ -200,7 +200,7 @@ struct Option {
     return *this;
   }
   Option& add_service(std::initializer_list<const char*> ss) {
-    services.insert(services.end(), ss.begin(), ss.end());
+    services.insert(services.end(), ss);
     return *this;
   }
   Option& add_see_also(const char* t) {
@@ -208,7 +208,7 @@ struct Option {
     return *this;
   }
   Option& add_see_also(std::initializer_list<const char*> ts) {
-    see_also.insert(see_also.end(), ts.begin(), ts.end());
+    see_also.insert(see_also.end(), ts);
     return *this;
   }
   Option& set_description(const char* new_desc) {
-- 
2.47.3