From 18634602ef9bccc1d6fea8d7a7976240797b0fcb Mon Sep 17 00:00:00 2001
From: Daniel Sipka <no1msd@gmail.com>
Date: Tue, 5 May 2015 08:49:42 +0200
Subject: [PATCH] simplify code

---
 src/template_type.cpp          | 12 ++++++++----
 src/template_type.hpp          |  1 +
 src/visitor/has_token.hpp      |  2 +-
 src/visitor/is_node_empty.hpp  |  2 +-
 src/visitor/render_node.hpp    |  2 +-
 src/visitor/render_section.hpp |  2 +-
 6 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/src/template_type.cpp b/src/template_type.cpp
index 53f6e81..355c4f6 100644
--- a/src/template_type.cpp
+++ b/src/template_type.cpp
@@ -74,10 +74,7 @@ void template_type::strip_whitespace() {
 
     if ((*it).eol()) {
       if (has_tag && !non_space) {
-        for (auto cur = line_begin; !(*cur).eol(); ++cur)
-          if ((*cur).token_type() == token::type::partial &&
-              cur != line_begin && (*(cur - 1)).ws_only())
-            (*cur).partial_prefix((*(cur - 1)).raw());
+        store_prefixes(line_begin);
 
         for (auto cur = line_begin; it != cur - 1;
              cur = (*cur).ws_only() ? tokens.erase(cur) : cur + 1)
@@ -89,3 +86,10 @@ void template_type::strip_whitespace() {
     }
   }
 }
+
+void template_type::store_prefixes(std::vector<token>::iterator beg) {
+  for (auto cur = beg; !(*cur).eol(); ++cur)
+    if ((*cur).token_type() == token::type::partial &&
+        cur != beg && (*(cur - 1)).ws_only())
+      (*cur).partial_prefix((*(cur - 1)).raw());
+}
diff --git a/src/template_type.hpp b/src/template_type.hpp
index 2a46b8d..4dfe826 100644
--- a/src/template_type.hpp
+++ b/src/template_type.hpp
@@ -21,6 +21,7 @@ class template_type {
   void strip_whitespace();
   void process_text(citer beg, citer end);
   void tokenize(const std::string& tmp);
+  void store_prefixes(std::vector<token>::iterator beg);
 };
 
 }
diff --git a/src/visitor/has_token.hpp b/src/visitor/has_token.hpp
index 2efb627..1d15443 100644
--- a/src/visitor/has_token.hpp
+++ b/src/visitor/has_token.hpp
@@ -12,7 +12,7 @@ class has_token: public boost::static_visitor<bool> {
   }
 
   template<class T>
-  inline bool operator()(const T& t) const {
+  bool operator()(const T& t) const {
     return token == ".";
   }
 
diff --git a/src/visitor/is_node_empty.hpp b/src/visitor/is_node_empty.hpp
index bfe3012..8119d4c 100644
--- a/src/visitor/is_node_empty.hpp
+++ b/src/visitor/is_node_empty.hpp
@@ -9,7 +9,7 @@ namespace mstch {
 class is_node_empty: public boost::static_visitor<bool> {
  public:
   template<class T>
-  inline bool operator()(const T& t) const {
+  bool operator()(const T& t) const {
     return false;
   }
 
diff --git a/src/visitor/render_node.hpp b/src/visitor/render_node.hpp
index 97037a6..63b7682 100644
--- a/src/visitor/render_node.hpp
+++ b/src/visitor/render_node.hpp
@@ -18,7 +18,7 @@ class render_node: public boost::static_visitor<std::string> {
   }
 
   template<class T>
-  inline std::string operator()(const T& t) const {
+  std::string operator()(const T& t) const {
     return "";
   }
 
diff --git a/src/visitor/render_section.hpp b/src/visitor/render_section.hpp
index be26435..25592ad 100644
--- a/src/visitor/render_section.hpp
+++ b/src/visitor/render_section.hpp
@@ -20,7 +20,7 @@ class render_section: public boost::static_visitor<std::string> {
   }
 
   template<class T>
-  inline std::string operator()(const T& t) const {
+  std::string operator()(const T& t) const {
     return render_context::push(ctx, t).render(section);
   }