]> sigrok.org Git - libsigrok.git/blobdiff - bindings/cxx/include/libsigrok/libsigrok.hpp
C++: Make StructureWrapper a recurring template, eliminating lots of casts.
[libsigrok.git] / bindings / cxx / include / libsigrok / libsigrok.hpp
index 546a4502e7fb6fe945c5c2070e01bbe24c13c94a..6260858274ade8db0a9aff8fa62fc6ea7a67ebb4 100644 (file)
@@ -125,8 +125,8 @@ public:
 };
 
 /* Base template for most classes which wrap a struct type from libsigrok. */
-template <class Parent, typename Struct> class SR_API StructureWrapper :
-       public enable_shared_from_this<StructureWrapper<Parent, Struct> >
+template <class Class, class Parent, typename Struct>
+class SR_API StructureWrapper
 {
 protected:
        /*  Parent object which owns this child object's underlying structure.
@@ -144,30 +144,49 @@ protected:
                references to both the parent and all its children are gone. */
        shared_ptr<Parent> parent;
 
+       /* Weak pointer for shared_from_this() implementation. */
+       weak_ptr<Class> weak_this;
+
 public:
-       shared_ptr<StructureWrapper<Parent, Struct> >
-       get_shared_pointer(Parent *parent)
+       /* Note, this implementation will create a new smart_ptr if none exists. */
+       shared_ptr<Class> shared_from_this()
        {
-               this->parent = static_pointer_cast<Parent>(parent->shared_from_this());
-               return shared_ptr<StructureWrapper<Parent, Struct> >(
-                       this, reset_parent);
+               shared_ptr<Class> shared;
+
+               if (!(shared = weak_this.lock()))
+               {
+                       shared = shared_ptr<Class>((Class *) this, reset_parent);
+                       weak_this = shared;
+               }
+
+               return shared;
        }
-       shared_ptr<StructureWrapper<Parent, Struct> >
-       get_shared_pointer(shared_ptr<Parent> parent)
+
+       shared_ptr<Class> get_shared_pointer(shared_ptr<Parent> parent)
        {
+               if (!parent)
+                       throw Error(SR_ERR_BUG);
                this->parent = parent;
-               return shared_ptr<StructureWrapper<Parent, Struct> >(
-                       this, reset_parent);
+               return shared_from_this();
+       }
+
+       shared_ptr<Class> get_shared_pointer(Parent *parent)
+       {
+               if (!parent)
+                       throw Error(SR_ERR_BUG);
+               return get_shared_pointer(parent->shared_from_this());
        }
 protected:
-       static void reset_parent(StructureWrapper<Parent, Struct> *object)
+       static void reset_parent(Class *object)
        {
+               if (!object->parent)
+                       throw Error(SR_ERR_BUG);
                object->parent.reset();
        }
 
        Struct *structure;
 
-       StructureWrapper<Parent, Struct>(Struct *structure) :
+       StructureWrapper<Class, Parent, Struct>(Struct *structure) :
                structure(structure)
        {
        }
@@ -242,7 +261,8 @@ protected:
 };
 
 /** A hardware driver provided by the library */
-class SR_API Driver : public StructureWrapper<Context, struct sr_dev_driver>
+class SR_API Driver :
+       public StructureWrapper<Driver, Context, struct sr_dev_driver>
 {
 public:
        /** Name of this driver. */
@@ -289,9 +309,7 @@ protected:
 };
 
 /** A generic device, either hardware or virtual */
-class SR_API Device :
-       public Configurable,
-       public StructureWrapper<Context, struct sr_dev_inst>
+class SR_API Device : public Configurable
 {
 public:
        /** Description identifying this device. */
@@ -313,7 +331,9 @@ public:
 protected:
        Device(struct sr_dev_inst *structure);
        ~Device();
+       virtual shared_ptr<Device> get_shared_from_this() = 0;
        shared_ptr<Channel> get_channel(struct sr_channel *ptr);
+       struct sr_dev_inst *structure;
        map<struct sr_channel *, Channel *> channels;
        map<string, ChannelGroup *> channel_groups;
        /** Deleter needed to allow shared_ptr use with protected destructor. */
@@ -331,7 +351,9 @@ protected:
 };
 
 /** A real hardware device, connected via a driver */
-class SR_API HardwareDevice : public Device
+class SR_API HardwareDevice :
+       public StructureWrapper<HardwareDevice, Context, struct sr_dev_inst>,
+       public Device
 {
 public:
        /** Driver providing this device. */
@@ -339,13 +361,15 @@ public:
 protected:
        HardwareDevice(Driver *driver, struct sr_dev_inst *structure);
        ~HardwareDevice();
+       shared_ptr<Device> get_shared_from_this();
        Driver *driver;
        friend class Driver;
        friend class ChannelGroup;
 };
 
 /** A channel on a device */
-class SR_API Channel : public StructureWrapper<Device, struct sr_channel>
+class SR_API Channel :
+       public StructureWrapper<Channel, Device, struct sr_channel>
 {
 public:
        /** Current name of this channel. */
@@ -374,7 +398,7 @@ protected:
 
 /** A group of channels on a device, which share some configuration */
 class SR_API ChannelGroup :
-       public StructureWrapper<Device, struct sr_channel_group>,
+       public StructureWrapper<ChannelGroup, Device, struct sr_channel_group>,
        public Configurable
 {
 public:
@@ -416,7 +440,8 @@ protected:
 };
 
 /** A stage in a trigger configuration */
-class SR_API TriggerStage : public StructureWrapper<Trigger, struct sr_trigger_stage>
+class SR_API TriggerStage :
+       public StructureWrapper<TriggerStage, Trigger, struct sr_trigger_stage>
 {
 public:
        /** Index number of this stage. */
@@ -440,7 +465,8 @@ protected:
 };
 
 /** A match condition in a trigger configuration  */
-class SR_API TriggerMatch : public StructureWrapper<TriggerStage, struct sr_trigger_match>
+class SR_API TriggerMatch :
+       public StructureWrapper<TriggerMatch, TriggerStage, struct sr_trigger_match>
 {
 public:
        /** Channel this condition matches on. */
@@ -642,9 +668,7 @@ class SR_API PacketPayload
 protected:
        PacketPayload();
        virtual ~PacketPayload() = 0;
-       shared_ptr<PacketPayload> get_shared_pointer(Packet *parent) {
-               return static_pointer_cast<PacketPayload>(get_shared_pointer(parent));
-       }
+       virtual shared_ptr<PacketPayload> get_shared_pointer(Packet *parent) = 0;
        /** Deleter needed to allow shared_ptr use with protected destructor. */
        class Deleter
        {
@@ -657,8 +681,9 @@ protected:
 };
 
 /** Payload of a datafeed header packet */
-class SR_API Header : public PacketPayload,
-       public StructureWrapper<Packet, const struct sr_datafeed_header>
+class SR_API Header :
+       public StructureWrapper<Header, Packet, const struct sr_datafeed_header>,
+       public PacketPayload
 {
 public:
        /* Feed version number. */
@@ -668,13 +693,14 @@ public:
 protected:
        Header(const struct sr_datafeed_header *structure);
        ~Header();
-       const struct sr_datafeed_header *structure;
+       shared_ptr<PacketPayload> get_shared_pointer(Packet *parent);
        friend class Packet;
 };
 
 /** Payload of a datafeed metadata packet */
-class SR_API Meta : public PacketPayload,
-       public StructureWrapper<Packet, const struct sr_datafeed_meta>
+class SR_API Meta :
+       public StructureWrapper<Meta, Packet, const struct sr_datafeed_meta>,
+       public PacketPayload
 {
 public:
        /* Mapping of (ConfigKey, value) pairs. */
@@ -682,14 +708,15 @@ public:
 protected:
        Meta(const struct sr_datafeed_meta *structure);
        ~Meta();
-       const struct sr_datafeed_meta *structure;
+       shared_ptr<PacketPayload> get_shared_pointer(Packet *parent);
        map<const ConfigKey *, Glib::VariantBase> config;
        friend class Packet;
 };
 
 /** Payload of a datafeed packet with logic data */
-class SR_API Logic : public PacketPayload,
-       public StructureWrapper<Packet, const struct sr_datafeed_logic>
+class SR_API Logic :
+       public StructureWrapper<Logic, Packet, const struct sr_datafeed_logic>,
+       public PacketPayload
 {
 public:
        /* Pointer to data. */
@@ -701,13 +728,14 @@ public:
 protected:
        Logic(const struct sr_datafeed_logic *structure);
        ~Logic();
-       const struct sr_datafeed_logic *structure;
+       shared_ptr<PacketPayload> get_shared_pointer(Packet *parent);
        friend class Packet;
 };
 
 /** Payload of a datafeed packet with analog data */
-class SR_API Analog : public PacketPayload,
-       public StructureWrapper<Packet, const struct sr_datafeed_analog>
+class SR_API Analog :
+       public StructureWrapper<Analog, Packet, const struct sr_datafeed_analog>,
+       public PacketPayload
 {
 public:
        /** Pointer to data. */
@@ -725,13 +753,13 @@ public:
 protected:
        Analog(const struct sr_datafeed_analog *structure);
        ~Analog();
-       const struct sr_datafeed_analog *structure;
+       shared_ptr<PacketPayload> get_shared_pointer(Packet *parent);
        friend class Packet;
 };
 
 /** An input format supported by the library */
 class SR_API InputFormat :
-       public StructureWrapper<Context, const struct sr_input_module>
+       public StructureWrapper<InputFormat, Context, const struct sr_input_module>
 {
 public:
        /** Name of this input format. */
@@ -777,11 +805,14 @@ protected:
 };
 
 /** A virtual device associated with an input */
-class SR_API InputDevice : public Device
+class SR_API InputDevice :
+       public StructureWrapper<InputDevice, Input, struct sr_dev_inst>,
+       public Device
 {
 protected:
        InputDevice(shared_ptr<Input> input, struct sr_dev_inst *sdi);
        ~InputDevice();
+       shared_ptr<Device> get_shared_from_this();
        shared_ptr<Input> input;
        /** Deleter needed to allow shared_ptr use with protected destructor. */
        class Deleter
@@ -820,12 +851,13 @@ protected:
                void operator()(Option *option) { delete option; }
        };
        friend class Deleter;
+       friend class InputFormat;
        friend class OutputFormat;
 };
 
 /** An output format supported by the library */
 class SR_API OutputFormat :
-       public StructureWrapper<Context, const struct sr_output_module>
+       public StructureWrapper<OutputFormat, Context, const struct sr_output_module>
 {
 public:
        /** Name of this output format. */