Improve getDecisionItem() and match() performance

Change-Id: I117a1d496cb709fc63958e0d826725fb01597daa

diff --git a/src/internal/policy_containers.hpp b/src/internal/policy_containers.hpp
index 5b0ddca..47f70f0 100644 (file)
--- a/src/internal/policy_containers.hpp
+++ b/src/internal/policy_containers.hpp
@@ -45,7 +45,7 @@ template <typename TI>
 class PolicySRBase : public PolicyBase<TI> {
 public:
        DecisionItem getDecisionItem(const typename TI::match_type &item) const {
-               for (auto i : *this) {
+               for (auto &i : *this) {
                        tslog::log_verbose("-read: ", i.getDecision(), " ", i, "\n");
 
                        if (item.match(i.getType(), i.getInterface(), i.getPath(),

diff --git a/src/internal/storage_backend_serialized.cpp b/src/internal/storage_backend_serialized.cpp
index fb5ccda..ae2b350 100644 (file)
--- a/src/internal/storage_backend_serialized.cpp
+++ b/src/internal/storage_backend_serialized.cpp
@@ -162,10 +162,18 @@ void StorageBackendSerialized::printContent() const {
        pimpl->printContent();
 }
 
+inline boost::string_ref s(const flatbuffers::String *str) {
+       return boost::string_ref(str->c_str(), str->size());
+}
+
 template <typename T, typename I>
 bool match(const T &match, const I *i) {
-       return match.match(makeMessageType(i->type()), i->interface()->c_str(), i->path()->c_str(),
-                       i->member()->c_str(), i->name()->c_str(), i->is_name_prefix());
+       return match.match(makeMessageType(i->type()),
+                                          s(i->interface()),
+                                          s(i->path()),
+                                          s(i->member()),
+                                          s(i->name()),
+                                          i->is_name_prefix());
 }
 
 template <> bool match(const ldp_xml_parser::MatchItemAccess &match, const FB::ItemAccess *item) {