review changes nr. 2 - changed 'index_info' from tuple to simple struct

Author: d-w-moore

indexing_utilities.cpp

@@ -299,13 +299,23 @@ namespace irods {
 
         };
 
-        class index_info final : public std::tuple<std::string,std::string,std::string> {
+        struct index_info
+        {
         public:
-            using string = std::string;
-            using tuple::tuple;
-            string& index_name()  { return std::get<0>(*this); }
-            string& index_type()  { return std::get<1>(*this); }
-            string& index_tech()  { return std::get<2>(*this); }
+            index_info ( const std::string & index_name_
+                        ,const std::string & index_type_
+                        ,const std::string & index_tech_ )
+                : index_name {index_name_}
+                 ,index_type {index_type_}
+                 ,index_tech {index_tech_}
+            {
+	    }
+	    bool operator< (const index_info & other) const { return index_name < other.index_name ||
+	                                                             index_type < other.index_type ||
+	                                                             index_tech < other.index_tech;  }
+            std::string index_name;
+            std::string index_type;
+            std::string index_tech;
         };
 
         // - Starting at _collection_name , recurse over every sub-element of the tree
@@ -441,16 +451,14 @@ namespace irods {
                                 //if (_index_type == "metadata" ) {
                                     auto info_set = idx_info_cache.accum( path.string() );
                                     for (auto info: info_set) {
-                                        irods::log( LOG_NOTICE, fmt::format("DWM - reindexing for perms - name/type/tech=[{}][{}][{}]",
-                                                                            info.index_name(), info.index_type(), info.index_tech()) );
                                         schedule_policy_event_for_object(
                                             policy_name,
                                             path.string(),
                                             _user_name,
                                             EMPTY_RESOURCE_NAME,
-                                            info.index_tech(),
-                                            info.index_name(),
-                                            info.index_type(),
+                                            info.index_tech,
+                                            info.index_name,
+                                            info.index_type,
                                             generate_delay_execution_parameters(),
                                             {},{},{},
                                             {{ "job_category_tag", unique_key }} );