c++: Reworked the event class 41/20541/1
authorLoïc Collignon <loic.collignon@iot.bzh>
Fri, 4 Jan 2019 15:05:10 +0000 (16:05 +0100)
committerLoïc Collignon <loic.collignon@iot.bzh>
Mon, 11 Mar 2019 12:40:40 +0000 (13:40 +0100)
Added move semantic and remove invalidation method as it's not used anymore
since we drop the support of APIv2.

Change-Id: If53840010d6f24d410712915051386190f55b504
Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>
include/afb/c++/binding-wrap.hpp

index 81122cb..6fdf0f6 100644 (file)
@@ -91,10 +91,13 @@ class event
 {
        afb_event_t event_;
 public:
-       event() { invalidate(); }
+       event();
        event(afb_event_t e);
        event(const event &other);
+       event(event &&other);
+       ~event();
        event &operator=(const event &other);
+       event &operator=(event &&other);
 
        operator afb_event_t() const;
        afb_event_t operator->() const;
@@ -102,8 +105,6 @@ public:
        operator bool() const;
        bool is_valid() const;
 
-       void invalidate();
-
        int broadcast(json_object *object) const;
        int push(json_object *object) const;
 
@@ -234,9 +235,13 @@ public:
 /*************************************************************************/
 
 /* events */
-inline event::event(afb_event_t e) : event_(e) { }
-inline event::event(const event &other) : event_(other.event_) { }
+inline event::event() : event_{nullptr} { }
+inline event::event(afb_event_t e) : event_{e} { }
+inline event::event(event &&other) : event_{other.event_} { other.event_ = nullptr; }
+inline event::event(const event &other) : event_{other.event_} { addref(); }
+inline event::~event() { unref(); }
 inline event &event::operator=(const event &other) { event_ = other.event_; return *this; }
+inline event &event::operator=(event &&other) { event_ = other.event_; other.event_ = nullptr; return *this;}
 
 inline event::operator afb_event_t() const { return event_; }
 inline afb_event_t event::operator->() const { return event_; }
@@ -244,12 +249,10 @@ inline afb_event_t event::operator->() const { return event_; }
 inline event::operator bool() const { return is_valid(); }
 inline bool event::is_valid() const { return afb_event_is_valid(event_); }
 
-inline void event::invalidate() { event_ = nullptr; }
-
 inline int event::broadcast(json_object *object) const { return afb_event_broadcast(event_, object); }
 inline int event::push(json_object *object) const { return afb_event_push(event_, object); }
 
-inline void event::unref() { afb_event_unref(event_); invalidate(); }
+inline void event::unref() { if (event_) { afb_event_unref(event_); } event_ = nullptr; }
 inline void event::addref() { afb_event_addref(event_); }
 inline const char *event::name() const { return afb_event_name(event_); }