make construct simpler

[wxWidgets.git] / src / common / object.cpp

diff --git a/src/common/object.cpp b/src/common/object.cpp
index c07c3c3c7c21e7092cb31007b8c123488eaf4064..622569f308941ca4aae398ac63dfd2a5d4101bc4 100644 (file)
--- a/src/common/object.cpp
+++ b/src/common/object.cpp
@@ -1,15 +1,16 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        object.cpp
+// Name:        src/common/object.cpp

 // Purpose:     wxObject implementation
 // Author:      Julian Smart
 // Purpose:     wxObject implementation
 // Author:      Julian Smart

-// Modified by:
+// Modified by: Ron Lee

 // Created:     04/01/98
 // RCS-ID:      $Id$
 // Created:     04/01/98
 // RCS-ID:      $Id$

-// Copyright:   (c) Julian Smart and Markus Holzem
-// Licence:     wxWindows license
+// Copyright:   (c) 1998 Julian Smart
+//              (c) 2001 Ron Lee <ron@debian.org>
+// Licence:     wxWindows licence

 /////////////////////////////////////////////////////////////////////////////
 
 /////////////////////////////////////////////////////////////////////////////
 

-#ifdef __GNUG__
+#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)

 #pragma implementation "object.h"
 #endif
 
 #pragma implementation "object.h"
 #endif
 


@@ -22,379 +23,277 @@
 
 #ifndef WX_PRECOMP
     #include "wx/hash.h"
 
 #ifndef WX_PRECOMP
     #include "wx/hash.h"

-    #if wxUSE_SERIAL
-        #include "wx/objstrm.h"
-        #include "wx/serbase.h"
-
-        // for error messages
-        #include "wx/log.h"
-        #include "wx/intl.h"
-    #endif // wxUSE_SERIAL
-#endif // WX_PRECOMP
+    #include "wx/object.h"
+#endif

 
 #include <string.h>
 
 #include <string.h>

-#include <assert.h>

 
 #if (defined(__WXDEBUG__) && wxUSE_MEMORY_TRACING) || wxUSE_DEBUG_CONTEXT
 #include "wx/memory.h"
 #endif
 
 #if defined(__WXDEBUG__) || wxUSE_DEBUG_CONTEXT
 
 #if (defined(__WXDEBUG__) && wxUSE_MEMORY_TRACING) || wxUSE_DEBUG_CONTEXT
 #include "wx/memory.h"
 #endif
 
 #if defined(__WXDEBUG__) || wxUSE_DEBUG_CONTEXT

-    // for wxObject::Dump
-    #include "wx/ioswrap.h"

     #if defined(__VISAGECPP__)
     #if defined(__VISAGECPP__)

-    // help with VA debugging
-        #define DEBUG_PRINTF(NAME)   { static int raz=0; \
-          printf( #NAME " %i\n",raz); fflush(stdout);       \
-           raz++;                                        \
-         }
+        #define DEBUG_PRINTF(NAME) { static int raz=0; \
+            printf( #NAME " %i\n",raz); fflush(stdout); raz++; }

     #else
         #define DEBUG_PRINTF(NAME)
     #endif
     #else
         #define DEBUG_PRINTF(NAME)
     #endif

+#endif // __WXDEBUG__ || wxUSE_DEBUG_CONTEXT
+
+// we must disable optimizations for VC.NET because otherwise its too eager
+// linker discards wxClassInfo objects in release build thus breaking many,
+// many things
+#if defined __VISUALC__ && __VISUALC__ >= 1300
+    #pragma optimize("", off)
+#endif
+
+#if wxUSE_EXTENDED_RTTI
+const wxClassInfo* wxObject::ms_classParents[] = { NULL } ;
+ wxObject* wxVariantToObjectConverterwxObject ( wxxVariant &data )
+{ return data.wxTEMPLATED_MEMBER_CALL(Get , wxObject*) ; }
+ wxObject* wxVariantOfPtrToObjectConverterwxObject ( wxxVariant &data )
+{ return &data.wxTEMPLATED_MEMBER_CALL(Get , wxObject) ; }
+ wxxVariant wxObjectToVariantConverterwxObject ( wxObject *data )
+ { return wxxVariant( dynamic_cast<wxObject*> (data)  ) ; }
+ wxClassInfo wxObject::ms_classInfo(ms_classParents , wxT("") , wxT("wxObject"),
+            (int) sizeof(wxObject),                              \
+            (wxObjectConstructorFn) 0   ,
+                       (wxPropertyInfo*) NULL,(wxHandlerInfo*) NULL,0 , 0 ,
+                       0 , wxVariantOfPtrToObjectConverterwxObject , wxVariantToObjectConverterwxObject , wxObjectToVariantConverterwxObject);
+ template<> void wxStringReadValue(const wxString & , wxObject * & ){assert(0) ;}
+ template<> void wxStringWriteValue(wxString & , wxObject* const & ){assert(0) ;}
+ template<> void wxStringReadValue(const wxString & , wxObject & ){assert(0) ;}
+ template<> void wxStringWriteValue(wxString & , wxObject const & ){assert(0) ;}
+ wxClassTypeInfo s_typeInfo(wxT_OBJECT_PTR , &wxObject::ms_classInfo , NULL , NULL , typeid(wxObject*).name() ) ; 
+ wxClassTypeInfo s_typeInfowxObject(wxT_OBJECT , &wxObject::ms_classInfo , NULL , NULL , typeid(wxObject).name() ) ; 
+#else
+wxClassInfo wxObject::ms_classInfo( wxT("wxObject"), 0, 0,
+                                        (int) sizeof(wxObject),
+                                        (wxObjectConstructorFn) 0 );
+#endif
+
+// restore optimizations
+#if defined __VISUALC__ && __VISUALC__ >= 1300
+    #pragma optimize("", on)

 #endif
 
 #endif
 

-wxClassInfo wxObject::sm_classwxObject((wxChar *) wxT("wxObject"), (wxChar *) NULL, (wxChar *) NULL, (int ) sizeof(wxObject), (wxObjectConstructorFn) NULL);
-wxClassInfo* wxClassInfo::sm_first = (wxClassInfo *) NULL;
-wxHashTable* wxClassInfo::sm_classTable = (wxHashTable*) NULL;
+wxClassInfo* wxClassInfo::sm_first = NULL;
+wxHashTable* wxClassInfo::sm_classTable = NULL;

 
 // These are here so we can avoid 'always true/false' warnings
 // by referring to these instead of TRUE/FALSE
 const bool wxTrue = TRUE;
 const bool wxFalse = FALSE;
 
 
 // These are here so we can avoid 'always true/false' warnings
 // by referring to these instead of TRUE/FALSE
 const bool wxTrue = TRUE;
 const bool wxFalse = FALSE;
 

-/*
- * wxWindows root object.
- */
-
-wxObject::wxObject()
-{
-    m_refData = (wxObjectRefData *) NULL;
-#if wxUSE_SERIAL
-    m_serialObj = (wxObject_Serialize *)NULL;
-#endif
-}
-
-wxObject::~wxObject()
-{
-    UnRef();
-#if wxUSE_SERIAL
-    if (m_serialObj)
-        delete m_serialObj;
-#endif
-}
-
-/*
- * Is this object a kind of (a subclass of) 'info'?
- * E.g. is wxWindow a kind of wxObject?
- * Go from this class to superclass, taking into account
- * two possible base classes.
- */
-
+// Is this object a kind of (a subclass of) 'info'?
+// E.g. is wxWindow a kind of wxObject?
+// Go from this class to superclass, taking into account
+// two possible base classes.

 bool wxObject::IsKindOf(wxClassInfo *info) const
 {
     wxClassInfo *thisInfo = GetClassInfo();
 bool wxObject::IsKindOf(wxClassInfo *info) const
 {
     wxClassInfo *thisInfo = GetClassInfo();

-    if (thisInfo)
-        return thisInfo->IsKindOf(info);
-    else
-        return FALSE;
+    return (thisInfo) ? thisInfo->IsKindOf(info) : FALSE ;

 }
 
 }
 

-wxObject *wxObject::Clone() const
-{
-    wxObject *object = GetClassInfo()->CreateObject();
-    CopyObject(*object);
-    return object;
-}
+#if defined(__WXDEBUG__) && wxUSE_MEMORY_TRACING && defined( new )
+       #undef new
+#endif

 
 

-#ifdef __WXDEBUG__
-void wxObject::CopyObject(wxObject& object_dest) const
-#else // !Debug
-void wxObject::CopyObject(wxObject& WXUNUSED(object_dest)) const
-#endif // Debug/!Debug
-{
-    wxASSERT(object_dest.GetClassInfo()->IsKindOf(GetClassInfo()));
-}

 
 

-#if wxUSE_STD_IOSTREAM && (defined(__WXDEBUG__) || wxUSE_DEBUG_CONTEXT)
-void wxObject::Dump(ostream& str)
+#ifdef _WX_WANT_NEW_SIZET_WXCHAR_INT
+void *wxObject::operator new ( size_t size, const wxChar *fileName, int lineNum )

 {
 {

-    if (GetClassInfo() && GetClassInfo()->GetClassName())
-        str << GetClassInfo()->GetClassName();
-    else
-        str << "unknown object class";
+    return wxDebugAlloc(size, (wxChar*) fileName, lineNum, TRUE);

 }
 #endif
 
 }
 #endif
 

-#if defined(__WXDEBUG__) && wxUSE_MEMORY_TRACING
-
-#ifdef new
-#undef new
-#endif
-
-void *wxObject::operator new (size_t size, wxChar * fileName, int lineNum)
+#ifdef _WX_WANT_DELETE_VOID
+void wxObject::operator delete ( void *buf )

 {
 {

-    return wxDebugAlloc(size, fileName, lineNum, TRUE);
+    wxDebugFree(buf);

 }
 }

+#endif

 
 

-#if defined(__VISAGECPP__)
-#  if __DEBUG_ALLOC__
-void wxObject::operator delete (void * buf,const char * _fname, size_t _line)
+#ifdef _WX_WANT_DELETE_VOID_CONSTCHAR_SIZET
+void wxObject::operator delete ( void *buf, const char *_fname, size_t _line )

 {
     wxDebugFree(buf);
 }
 {
     wxDebugFree(buf);
 }

-#  endif  //__DEBUG_ALLOC__
-#else
-void wxObject::operator delete (void * buf)
+#endif
+
+#ifdef _WX_WANT_DELETE_VOID_WXCHAR_INT
+void wxObject::operator delete ( void *buf, const wxChar *WXUNUSED(fileName), int WXUNUSED(lineNum) )

 {
 {

-    wxDebugFree(buf);
+     wxDebugFree(buf);

 }
 }

-#endif // __VISAGECPP__
+#endif

 
 

-// VC++ 6.0
-#if defined(__VISUALC__) && (__VISUALC__ >= 1200)
-void wxObject::operator delete(void* pData, wxChar* /* fileName */, int /* lineNum */)
+#ifdef _WX_WANT_ARRAY_NEW_SIZET_WXCHAR_INT
+void *wxObject::operator new[] ( size_t size, const wxChar* fileName, int lineNum )

 {
 {

-    ::operator delete(pData);
+    return wxDebugAlloc(size, (wxChar*) fileName, lineNum, TRUE, TRUE);

 }
 #endif
 
 }
 #endif
 

-// Cause problems for VC++ - crashes
-#if (!defined(__VISUALC__) && wxUSE_ARRAY_MEMORY_OPERATORS ) || defined(__MWERKS__)
-void * wxObject::operator new[] (size_t size, wxChar * fileName, int lineNum)
+#ifdef _WX_WANT_ARRAY_DELETE_VOID
+void wxObject::operator delete[] ( void *buf )

 {
 {

-    return wxDebugAlloc(size, fileName, lineNum, TRUE, TRUE);
+    wxDebugFree(buf, TRUE);

 }
 }

+#endif

 
 

-void wxObject::operator delete[] (void * buf)
+#ifdef _WX_WANT_ARRAY_DELETE_VOID_WXCHAR_INT
+void wxObject::operator delete[] (void * buf, const wxChar*  WXUNUSED(fileName), int WXUNUSED(lineNum) )

 {
     wxDebugFree(buf, TRUE);
 }
 #endif
 
 {
     wxDebugFree(buf, TRUE);
 }
 #endif
 

-#endif

 
 

-/*
- * Class info: provides run-time class type information.
- */
+// ----------------------------------------------------------------------------
+// wxClassInfo
+// ----------------------------------------------------------------------------

 
 

-wxClassInfo::wxClassInfo(const wxChar *cName,
-                         const wxChar *baseName1,
-                         const wxChar *baseName2,
-                         int sz,
-                         wxObjectConstructorFn constr)
+wxClassInfo::~wxClassInfo()

 {
 {

-    m_className = cName;
-    m_baseClassName1 = baseName1;
-    m_baseClassName2 = baseName2;
-
-    m_objectSize = sz;
-    m_objectConstructor = constr;
-
-    m_next = sm_first;
-    sm_first = this;
+    // remove this object from the linked list of all class infos: if we don't
+    // do it, loading/unloading a DLL containing static wxClassInfo objects is
+    // not going to work
+    if ( this == sm_first )
+    {
+        sm_first = m_next;
+    }
+    else
+    {
+        wxClassInfo *info = sm_first;
+        while (info)
+        {
+            if ( info->m_next == this )
+            {
+                info->m_next = m_next;
+                break;
+            }

 
 

-    m_baseInfo1 = (wxClassInfo *) NULL;
-    m_baseInfo2 = (wxClassInfo *) NULL;
+            info = info->m_next;
+        }
+    }
+       Unregister();

 }
 
 }
 

-wxObject *wxClassInfo::CreateObject()
+wxClassInfo *wxClassInfo::FindClass(const wxChar *className)

 {
 {

-    if (m_objectConstructor)
-        return (wxObject *)(*m_objectConstructor)();
+    if ( sm_classTable )
+    {
+        return (wxClassInfo *)wxClassInfo::sm_classTable->Get(className);
+    }

     else
     else

-        return (wxObject *) NULL;
-}
-
-wxClassInfo *wxClassInfo::FindClass(const wxChar *c)
-{
-    wxClassInfo *p = sm_first;
-    while (p)

     {
     {

-        if ( wxStrcmp(p->GetClassName(), c) == 0 )
-            break;
+        for ( wxClassInfo *info = sm_first; info ; info = info->m_next )
+        {
+            if ( wxStrcmp(info->GetClassName(), className) == 0 )
+                return info;
+        }

 
 

-        p = p->m_next;
+        return NULL;

     }
     }

-
-    return p;

 }
 
 }
 

-// Climb upwards through inheritance hierarchy.
-// Dual inheritance is catered for.
-bool wxClassInfo::IsKindOf(const wxClassInfo *info) const
+void wxClassInfo::CleanUp()

 {
 {

-    if (info == NULL)
-        return FALSE;
-
-    if (this == info)
-        return TRUE;
-
-    if (m_baseInfo1)
-        if (m_baseInfo1->IsKindOf(info))
-            return TRUE;
-
-    if (m_baseInfo2)
-        return m_baseInfo2->IsKindOf(info);
-
-    return FALSE;
+    if ( sm_classTable )
+    {
+        delete sm_classTable;
+        sm_classTable = NULL;
+    }

 }
 
 }
 

-// Set pointers to base class(es) to speed up IsKindOf
-void wxClassInfo::InitializeClasses()
+void wxClassInfo::Register()

 {
 {

+    if ( !sm_classTable )
+    {
+        sm_classTable = new wxHashTable(wxKEY_STRING);
+    }
+

     // using IMPLEMENT_DYNAMIC_CLASS() macro twice (which may happen if you
     // link any object module twice mistakenly) will break this function
     // because it will enter an infinite loop and eventually die with "out of
     // memory" - as this is quite hard to detect if you're unaware of this,
     // try to do some checks here
     // using IMPLEMENT_DYNAMIC_CLASS() macro twice (which may happen if you
     // link any object module twice mistakenly) will break this function
     // because it will enter an infinite loop and eventually die with "out of
     // memory" - as this is quite hard to detect if you're unaware of this,
     // try to do some checks here

-#ifdef __WXDEBUG__
-    // more classes than we'll ever have
-    static const size_t nMaxClasses = 10000;
-    size_t nClass = 0;
-#endif // Debug
-
-    wxClassInfo::sm_classTable = new wxHashTable(wxKEY_STRING);
+    wxASSERT_MSG( sm_classTable->Get(m_className) == NULL,
+                  _T("class already in RTTI table - have you used IMPLEMENT_DYNAMIC_CLASS() twice (may be by linking some object module(s) twice)?") );
+    
+    sm_classTable->Put(m_className, (wxObject *)this);
+}

 
 

-    // Index all class infos by their class name
-    wxClassInfo *info = sm_first;
-    while (info)
+void wxClassInfo::Unregister()
+{
+    if ( sm_classTable )

     {
     {

-        if (info->m_className)
+        sm_classTable->Delete(m_className);
+        if ( sm_classTable->GetCount() == 0 )

         {
         {

-            wxASSERT_MSG( ++nClass < nMaxClasses,
-                          _T("an infinite loop detected - have you used IMPLEMENT_DYNAMIC_CLASS() twice (may be by linking some object module(s) twice)?") );
-
-            sm_classTable->Put(info->m_className, (wxObject *)info);
+            delete sm_classTable;
+            sm_classTable = NULL;

         }
         }

-
-        info = info->m_next;

     }
     }

-
-    // Set base pointers for each wxClassInfo
-    info = sm_first;
-    while (info)
-    {
-        if (info->GetBaseClassName1())
-            info->m_baseInfo1 = (wxClassInfo *)sm_classTable->Get(info->GetBaseClassName1());
-        if (info->GetBaseClassName2())
-            info->m_baseInfo2 = (wxClassInfo *)sm_classTable->Get(info->GetBaseClassName2());
-        info = info->m_next;
-    }
-}
-
-void wxClassInfo::CleanUpClasses()
-{
-    delete wxClassInfo::sm_classTable;
-    wxClassInfo::sm_classTable = NULL;

 }
 
 wxObject *wxCreateDynamicObject(const wxChar *name)
 {
 #if defined(__WXDEBUG__) || wxUSE_DEBUG_CONTEXT
 }
 
 wxObject *wxCreateDynamicObject(const wxChar *name)
 {
 #if defined(__WXDEBUG__) || wxUSE_DEBUG_CONTEXT

- DEBUG_PRINTF(wxObject *wxCreateDynamicObject)
+    DEBUG_PRINTF(wxObject *wxCreateDynamicObject)

 #endif
 
 #endif
 

-    if (wxClassInfo::sm_classTable)
+    if ( wxClassInfo::sm_classTable )

     {
         wxClassInfo *info = (wxClassInfo *)wxClassInfo::sm_classTable->Get(name);
     {
         wxClassInfo *info = (wxClassInfo *)wxClassInfo::sm_classTable->Get(name);

-        if (!info)
-            return (wxObject *)NULL;
-
-        return info->CreateObject();
+        return info ? info->CreateObject() : NULL;

     }
     }

-    else
+    else // no sm_classTable yet

     {
     {

-        wxClassInfo *info = wxClassInfo::sm_first;
-        while (info)
+        for ( wxClassInfo *info = wxClassInfo::sm_first;
+              info;
+              info = info->m_next )

         {
             if (info->m_className && wxStrcmp(info->m_className, name) == 0)
                 return info->CreateObject();
         {
             if (info->m_className && wxStrcmp(info->m_className, name) == 0)
                 return info->CreateObject();

-            info = info->m_next;

         }
         }

-        return (wxObject*) NULL;
+
+        return NULL;

     }
 }
 
     }
 }
 

-#if wxUSE_SERIAL

 
 

-#include "wx/serbase.h"
-#include "wx/dynlib.h"
+// ----------------------------------------------------------------------------
+// wxObject
+// ----------------------------------------------------------------------------

 
 

-wxObject* wxCreateStoredObject( wxInputStream &stream )
+// Initialize ref data from another object (needed for copy constructor and
+// assignment operator)
+void wxObject::InitFrom(const wxObject& other)

 {
 {

-    wxObjectInputStream obj_s(stream);
-    return obj_s.LoadObject();
-};
-
-void wxObject::StoreObject( wxObjectOutputStream& stream )
-{
-#if defined(__WXDEBUG__) || wxUSE_DEBUG_CONTEXT
- DEBUG_PRINTF(wxObject::StoreObject)
-#endif
-
-    wxString obj_name = wxString(GetClassInfo()->GetClassName()) + "_Serialize";
-    wxLibrary *lib = wxTheLibraries.LoadLibrary("wxserial");
-
-    if (!lib) {
-        wxLogError(_("Can't load wxSerial dynamic library."));
-        return;
-    }
-    if (!m_serialObj) {
-        m_serialObj = (WXSERIAL(wxObject) *)lib->CreateObject( obj_name );
-
-        if (!m_serialObj) {
-            wxLogError(_("Can't find the serialization object '%s' "
-                        "for the object '%s'."),
-                    obj_name.c_str(),
-                    GetClassInfo()->GetClassName());
-            return;
-        }
-        m_serialObj->SetObject(this);
-    }
-
-    m_serialObj->StoreObject(stream);
+    m_refData = other.m_refData;
+    if ( m_refData )
+        m_refData->m_count++;

 }
 
 }
 

-void wxObject::LoadObject( wxObjectInputStream& stream )
+void wxObject::Ref(const wxObject& clone)

 {
 #if defined(__WXDEBUG__) || wxUSE_DEBUG_CONTEXT
 {
 #if defined(__WXDEBUG__) || wxUSE_DEBUG_CONTEXT

- DEBUG_PRINTF(wxObject::LoadObject)
+    DEBUG_PRINTF(wxObject::Ref)

 #endif
 
 #endif
 

-    wxString obj_name = wxString(GetClassInfo()->GetClassName()) + "_Serialize";
-    wxLibrary *lib = wxTheLibraries.LoadLibrary("wxserial");
-
-    if (!m_serialObj) {
-        m_serialObj = (WXSERIAL(wxObject) *)lib->CreateObject( obj_name );
-
-        if (!m_serialObj) {
-            wxLogError(_("Can't find the serialization object '%s' "
-                        "for the object '%s'."),
-                    obj_name.c_str(),
-                    GetClassInfo()->GetClassName());
-            return;
-        }
-        m_serialObj->SetObject(this);
-    }
-
-    m_serialObj->LoadObject(stream);
-}
-
-#endif // wxUSE_SERIAL
-
-/*
- * wxObject: cloning of objects
- */
+    // nothing to be done
+    if (m_refData == clone.m_refData)
+        return;

 
 

-void wxObject::Ref(const wxObject& clone)
-{
-#if defined(__WXDEBUG__) || wxUSE_DEBUG_CONTEXT
- DEBUG_PRINTF(wxObject::Ref)
-#endif
-     // delete reference to old data
+    // delete reference to old data

     UnRef();
     UnRef();

+

     // reference new data
     // reference new data

-    if (clone.m_refData) {
+    if ( clone.m_refData )
+    {

         m_refData = clone.m_refData;
         ++(m_refData->m_count);
     }
         m_refData = clone.m_refData;
         ++(m_refData->m_count);
     }


@@ -408,18 +307,44 @@ void wxObject::UnRef()
 
         if ( !--m_refData->m_count )
             delete m_refData;
 
         if ( !--m_refData->m_count )
             delete m_refData;

-        m_refData = (wxObjectRefData *) NULL;
+        m_refData = NULL;

     }
 }
 
     }
 }
 

-/*
- * wxObjectData
- */
+void wxObject::AllocExclusive()
+{
+    if ( !m_refData )
+    {
+        m_refData = CreateRefData();
+    }
+    else if ( m_refData->GetRefCount() > 1 )
+    {
+        // note that ref is not going to be destroyed in this case
+        const wxObjectRefData* ref = m_refData;
+        UnRef();
+
+        // ... so we can still access it
+        m_refData = CloneRefData(ref);
+    }
+    //else: ref count is 1, we are exclusive owners of m_refData anyhow

 
 

-wxObjectRefData::wxObjectRefData(void) : m_count(1)
+    wxASSERT_MSG( m_refData && m_refData->GetRefCount() == 1,
+                  _T("wxObject::AllocExclusive() failed.") );
+}
+
+wxObjectRefData *wxObject::CreateRefData() const

 {
 {

+    // if you use AllocExclusive() you must override this method
+    wxFAIL_MSG( _T("CreateRefData() must be overridden if called!") );
+
+    return NULL;

 }
 
 }
 

-wxObjectRefData::~wxObjectRefData()
+wxObjectRefData *
+wxObject::CloneRefData(const wxObjectRefData * WXUNUSED(data)) const

 {
 {

+    // if you use AllocExclusive() you must override this method
+    wxFAIL_MSG( _T("CloneRefData() must be overridden if called!") );
+
+    return NULL;

 }
 }