From 66609cfba2b581be52de6096193751d1bb4ec3c3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?louiz=E2=80=99?= <louiz@louiz.org>
Date: Sun, 22 May 2016 12:52:05 +0200
Subject: Remove all usage of std::list

---
 louloulibs/network/tcp_socket_handler.hpp |  2 +-
 louloulibs/utils/timed_events.cpp         | 11 +----------
 louloulibs/utils/timed_events.hpp         | 18 +++++++++---------
 3 files changed, 11 insertions(+), 20 deletions(-)

(limited to 'louloulibs')

diff --git a/louloulibs/network/tcp_socket_handler.hpp b/louloulibs/network/tcp_socket_handler.hpp
index dd4e82e..55f4113 100644
--- a/louloulibs/network/tcp_socket_handler.hpp
+++ b/louloulibs/network/tcp_socket_handler.hpp
@@ -180,7 +180,7 @@ private:
   /**
    * Where data is added, when we want to send something to the client.
    */
-  std::list<std::string> out_buf;
+  std::vector<std::string> out_buf;
   /**
    * DNS resolver
    */
diff --git a/louloulibs/utils/timed_events.cpp b/louloulibs/utils/timed_events.cpp
index 930380b..6b936c4 100644
--- a/louloulibs/utils/timed_events.cpp
+++ b/louloulibs/utils/timed_events.cpp
@@ -20,15 +20,6 @@ TimedEvent::TimedEvent(std::chrono::milliseconds&& duration,
 {
 }
 
-TimedEvent::TimedEvent(TimedEvent&& other):
-  time_point(std::move(other.time_point)),
-  callback(std::move(other.callback)),
-  repeat(other.repeat),
-  repeat_delay(std::move(other.repeat_delay)),
-  name(std::move(other.name))
-{
-}
-
 bool TimedEvent::is_after(const TimedEvent& other) const
 {
   return this->is_after(other.time_point);
@@ -47,7 +38,7 @@ std::chrono::milliseconds TimedEvent::get_timeout() const
   return std::chrono::duration_cast<std::chrono::milliseconds>(this->time_point - now);
 }
 
-void TimedEvent::execute()
+void TimedEvent::execute() const
 {
   this->callback();
 }
diff --git a/louloulibs/utils/timed_events.hpp b/louloulibs/utils/timed_events.hpp
index c3dfc40..70e2eff 100644
--- a/louloulibs/utils/timed_events.hpp
+++ b/louloulibs/utils/timed_events.hpp
@@ -4,7 +4,7 @@
 #include <functional>
 #include <string>
 #include <chrono>
-#include <list>
+#include <vector>
 
 using namespace std::literals::chrono_literals;
 
@@ -30,12 +30,12 @@ public:
   explicit TimedEvent(std::chrono::milliseconds&& duration,
                       std::function<void()> callback, const std::string& name="");
 
-  explicit TimedEvent(TimedEvent&&);
+  explicit TimedEvent(TimedEvent&&) = default;
+  TimedEvent& operator=(TimedEvent&&) = default;
   ~TimedEvent() = default;
 
   TimedEvent(const TimedEvent&) = delete;
   TimedEvent& operator=(const TimedEvent&) = delete;
-  TimedEvent& operator=(TimedEvent&&) = delete;
 
   /**
    * Whether or not this event happens after the other one.
@@ -48,7 +48,7 @@ public:
    * returned value is 0 instead. The value cannot then be negative.
    */
   std::chrono::milliseconds get_timeout() const;
-  void execute();
+  void execute() const;
   const std::string& get_name() const;
 
 private:
@@ -59,22 +59,22 @@ private:
   /**
    * The function to execute.
    */
-  const std::function<void()> callback;
+  std::function<void()> callback;
   /**
    * Whether or not this events repeats itself until it is destroyed.
    */
-  const bool repeat;
+  bool repeat;
   /**
    * This value is added to the time_point each time the event is executed,
    * if repeat is true. Otherwise it is ignored.
    */
-  const std::chrono::milliseconds repeat_delay;
+  std::chrono::milliseconds repeat_delay;
   /**
    * A name that is used to identify that event. If you want to find your
    * event (for example if you want to cancel it), the name should be
    * unique.
    */
-  const std::string name;
+  std::string name;
 };
 
 /**
@@ -128,7 +128,7 @@ public:
   std::size_t size() const;
 
 private:
-  std::list<TimedEvent> events;
+  std::vector<TimedEvent> events;
   explicit TimedEventsManager() = default;
 };
 
-- 
cgit v1.2.3