Changeset 527 for library/bdm/base/user_info.h

Timestamp:

08/13/09 15:58:32 (15 years ago)

Author:

vbarta

Message:

using shared_ptr in UI (optionally so far; loading & saving Array<T *> still works but should be phased out); testsuite run leaks down from 8822 to 480 bytes

Files:

• library/bdm/base/user_info.h (modified) (8 diffs)

library/bdm/base/user_info.h

@@ -24 +24 @@
 #include "libconfig/libconfig.h++"
 #include "../bdmroot.h"
+#include "../shared_ptr.h"
 #include "itpp/itbase.h"
 
@@ -253 +254 @@
         static void assert_type ( const Setting &element, Setting::Type type );
 
-        //! Method assembling a typeless instance, it is implemented in descendant class ParticularUI<T>
+        /*!
+          \brief Method constructing a configured instance
+
+          The returned pointer must be allocated using operator new
+          (it's deleted at the end of its life cycle). The method is
+          implemented in descendant class ParticularUI<T>, which knows
+          the correct type T.
+        */
         virtual root* new_instance() const = 0;
 
@@ -274 +282 @@
         //! This method converts a Setting into a real scalar
         static void from_setting ( double &real, const Setting &element );
+
         //! This method converts a Setting into a class T descendant
         template<class T> static void from_setting ( T* &instance, const Setting &element ) {
@@ -287 +296 @@
                 const UI& related_UI = MappedUI::retrieve_ui ( class_name );
 
-                root* typeless_instance = related_UI.new_instance();
+                root *typeless_instance = related_UI.new_instance();
+                it_assert_debug ( typeless_instance, "UI::new_instance failed" );
 
                 instance = dynamic_cast<T*> ( typeless_instance );
-                if ( !instance )
+                if ( !instance ) {
+                        delete typeless_instance;
                         throw UIException ( "class " + class_name + " is not a descendant of the desired output class. Try to call the UI::build<T> function with a different type parameter.", link.result );
+                }
 
                 try {
@@ -301 +313 @@
         }
 
+        //! This method converts a Setting into a descendant of class
+        //! T, wrapped in an instance of shared_ptr<T> .
+        template<class T>
+        static void from_setting ( shared_ptr<T> &instance, const Setting &element ) {
+                T *tmp_inst = 0;
+                from_setting ( tmp_inst, element );
+                it_assert_debug ( tmp_inst, "UI::from_setting failed" );
+                instance = tmp_inst;
+        }
+
         //! This methods converts a Setting into a new templated array of type Array<T>
         template<class T> static void from_setting ( Array<T> &array_to_load, const Setting &element ) {
@@ -320 +342 @@
         //! The exception can help to find the place where the template is misused and also to correct it.
         template<class T> static void from_setting ( T &variable_to_load, const Setting &element ) {
-                throw UIException ( "from_setting is not implemented for this type", element );
+                std::string msg = "from_setting is not implemented for type ";
+                msg += typeid(T).name();
+                throw UIException ( msg, element );
         }
 
@@ -341 +365 @@
         //! The new instance of type T* is constructed and initialized with values stored in the Setting element[name]
         //!
-        //! If there is not any sub-element named #name, the null pointer is returned.
-        template<class T> static T* build ( const Setting &element, const string &name, SettingPresence settingPresence = optional ) {
+        //! If there is not any sub-element named #name and settingPresence is #optional, an empty shared_ptr<T> is returned. When settingPresence is #compulsory, the returned shared_ptr<T> is never empty (an exception is thrown when the object isn't found).
+        template<class T>
+        static shared_ptr<T> build ( const Setting &element, const string &name, SettingPresence settingPresence = optional ) {
                 if ( !element.exists ( name ) ) {
                         if ( settingPresence == optional )
-                                return NULL;
+                                return shared_ptr<T>();
                         else
                                 throw UIException ( "the compulsory Setting named \"" + name + "\" is missing", element );
                 }
 
-                T* instance;
+                shared_ptr<T> instance;
                 from_setting<T> ( instance, to_child_setting ( element, name ) );
                 return instance;
@@ -483 +508 @@
         }
 
+        template< class T> static void save ( const shared_ptr<T> &instance, Setting &element, const string &name = "" ) {
+                save<T> ( instance.get(), element, name );
+        }
+
         //! An Array<T> instance is stored in the new child Setting appended to the passed element
         template<class T> static void save ( const Array<T> &array_to_save, Setting &element, const string &name = "" ) {