X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;f=kernel%2Fsrc%2FbbtkPackage.cxx;h=94286b37811f8f99a3cc8b17a8b1ddbd51c0636f;hb=4ad5b5ee44357ad873bc8c43230defb6d0a79879;hp=b24272152df64d0fa61b3a7d90c45efe5289823e;hpb=c2a4b1893412e50a3d9abff221938a2d16c4a7cb;p=bbtk.git

diff --git a/kernel/src/bbtkPackage.cxx b/kernel/src/bbtkPackage.cxx
index b242721..94286b3 100644
--- a/kernel/src/bbtkPackage.cxx
+++ b/kernel/src/bbtkPackage.cxx
@@ -3,8 +3,8 @@
   Program:   bbtk
   Module:    $RCSfile: bbtkPackage.cxx,v $
   Language:  C++
-  Date:      $Date: 2008/04/09 11:16:57 $
-  Version:   $Revision: 1.12 $
+  Date:      $Date: 2008/04/18 12:59:15 $
+  Version:   $Revision: 1.13 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See doc/license.txt or
@@ -28,6 +28,30 @@
 
 namespace bbtk
 {
+  
+
+
+  //==========================================================================
+  /// Creates a new package
+  Package::Pointer Package::New(const std::string& name,
+				const std::string& author,
+				const std::string& description,
+				const std::string& version,
+				const std::string& BBTKVersion) 
+  {
+    bbtkDebugMessage("object",1,"##> Package::New('"<<name<<"',...)"
+		     <<bbtkendl);
+    Package::Pointer p = MakePointer(new Package(name,
+					       author,
+					       description,
+					       version,
+					       BBTKVersion));
+    bbtkDebugMessage("object",2,"<## Package::New('"<<name<<"',...)"
+		     <<bbtkendl);
+    return p;
+  }
+  //==========================================================================
+
   //==========================================================================
   /// Ctor with the name of the package
   Package::Package(const std::string& name,
@@ -36,12 +60,14 @@ namespace bbtk
 		   const std::string& version,
 		   const std::string& BBTKVersion) 
     :
+    mDynamicLibraryHandler(0),
     mName(name),
     mAuthor(author),
     mDescription(description),
-    mVersion(version),
-    mBBTKVersion(BBTKVersion)
+    mVersion(version)
   {
+    bbtkDebugMessage("object",2,"==> Package::Package('"<<name<<"',...)"
+		     <<bbtkendl);
     std::string default_doc_dir = ConfigurationFile::GetInstance().Get_default_temp_dir();
     char c = default_doc_dir.c_str()[strlen(default_doc_dir.c_str())-1];
     std::string url = default_doc_dir; 
@@ -62,7 +88,9 @@ namespace bbtk
 
     //    std::cout  << "   url=["<<url<<"]"<<std::endl;
     //    std::cout  << "relurl=["<<relurl<<"]"<<std::endl;
-    bbtkDebugMessage("Kernel",7,"Package::Package(\""<<name<<"\")"<<bbtkendl);
+    bbtkDebugMessage("object",2,"<== Package::Package('"<<name<<"',...) OK"
+		     <<bbtkendl);
+
   }
   //==========================================================================
 
@@ -72,34 +100,542 @@ namespace bbtk
   /// Dtor
   Package::~Package()
   {
-    bbtkDebugMessageInc("Kernel",7,"Package::~Package(\""<<mName<<"\")"<<bbtkendl);
-    BlackBoxMapType::const_iterator i;
-    for (i=mBlackBoxMap.begin();
-         i!=mBlackBoxMap.end();
-       ++i) 
-     {
-       i->second->UnReference();
-     } 
-    // Adaptors are also stored in the black box map : hence already deleted
+    bbtkDebugMessage("object",2,"==> Package::~Package(\""<<mName<<"\")"<<bbtkendl);
+    bbtkDebugMessage("object",2,"<== Package::~Package(\""<<mName<<"\")"<<bbtkendl);
+  }
+  //==========================================================================
+
+
+  //==========================================================================
+  void PackageReleaseBlackBoxDescriptorInternal(Package::WeakPointer pack,
+						const std::string& descname)
+  {
+    // Try to release descriptor
+    bbtkDebugMessage("package",3,"--- Trying to release descriptor '"
+		     <<descname<<"'"<<bbtkendl);
+    std::string packname = pack.lock()->GetName();
+
+    Package::BlackBoxMapType::iterator desc = 
+      pack.lock()->GetBlackBoxMap().find(descname);
+    if (desc ==  pack.lock()->GetBlackBoxMap().end())
+      {
+	bbtkDebugMessage("package",3,
+			 "    Descriptor has already been released"
+			 <<bbtkendl);
+	return;
+      }
+    //    bbtkDebugMessage("package",3,
+    //		     "    Trying unreferencing it ... "<<std::endl);
+    BlackBoxDescriptor::WeakPointer pdesc = desc->second;
+    desc->second.reset();
+    // if it is dead : remove it 
+    if (pdesc.expired()) 
+      {
+	bbtkDebugMessage("package",2,"   '"<<packname<<"::"<<descname<<"' Descriptor expired"<<bbtkendl);
+	if (pack.expired()) 
+	  {
+	    bbtkDebugMessage("package",2,
+			     "    ... and caused its package death"
+			     <<bbtkendl);
+	    return;
+	  }
+	desc = pack.lock()->GetBlackBoxMap().find(descname);
+	if (desc !=  pack.lock()->GetBlackBoxMap().end())
+	  pack.lock()->GetBlackBoxMap().erase(desc);
+      }
+    else 
+      {
+	bbtkDebugMessage("package",3,"    ... Descriptor still alive ("
+			 <<pdesc.use_count()<<" refs)"
+			 <<bbtkendl);
+	pack.lock()->GetBlackBoxMap()[descname] = pdesc.lock();
+      }
+  }
+  //==========================================================================
+
+
+  //==========================================================================
+  /// Release
+  void Package::Release(Package::WeakPointer pack)
+  {
+    std::string packname = pack.lock()->mName;
+    bbtkDebugMessage("package",1,"==> Package::Release('"<<
+		     packname<<"')"<<bbtkendl);
+
+    long dyn = pack.lock()->mDynamicLibraryHandler ? 1:0; 
+    long ndesc = pack.lock()->GetBlackBoxMap().size();
+    long nrefs = pack.use_count();
+
+    bbtkDebugMessage("package",2," "<<nrefs<<" refs / "
+		     <<ndesc<<" descr / dyn="
+		     <<dyn<<std::endl);
+
+    // A package is "free" from any external reference iff :
+    // i) It is not dynamically loaded and nrefs == ndesc 
+    // (each desc references its package) or
+    // ii) It is dynamically loaded and nrefs == ndesc + 1
+    // (A dynamic library holds a static pointer on the package it contains
+    //  which is allocated when the PACKAGENAMEGetPackage() func is called,
+    //  and descallocated (reset) by PACKAGENAMEDeletePackage())
+    if (nrefs == ndesc + dyn) 
+      {
+	bbtkDebugMessage("package",2,
+			 " -> No more external ref : checking descriptors"
+			 <<bbtkendl);
+	// We must take care that removing refs on descriptors 
+	// can lead to their deletion which can in turn unref 
+	// internal boxes which can release their descriptors hence 
+	// call Package::ReleaseBlackBoxDescriptor 
+	// As a consequence during descriptors release :
+	// 1) The map can change dynamically : we cannot iterate over it 
+	//    as any iterator can become invalid
+	// 2) The package can auto-destruct : we must test its existence 
+	//    after each release
+	// We must also take care of not locking the package pointer 
+	// or any ref count check in Package::ReleaseBlackBoxDescriptor
+	// would be wrong
+
+	// The list of descriptors names at start
+	std::vector<std::string> descnamelist;
+	BlackBoxMapType::iterator i;
+	for (i=pack.lock()->mBlackBoxMap.begin();
+	     i!= pack.lock()->mBlackBoxMap.end();
+	     ++i)
+	  descnamelist.push_back(i->first);
+
+	// Iterator over the initial names
+	std::vector<std::string>::iterator descname;
+	for (descname=descnamelist.begin();
+	     descname!=descnamelist.end();
+	     ++descname)
+	  {
+	    // Is package still alive ?
+	    if (pack.expired()) 
+	      {
+		bbtkDebugMessage("package",2,"--- Package::Release('"<<
+				 packname
+				 <<"') : package expired during release : bailing out"<<bbtkendl);
+		break;
+	      }
+	    
+	    PackageReleaseBlackBoxDescriptorInternal(pack,*descname);
+	    
+	  }
+
+	//
+	UnLoadDynamicLibrary(pack);
+	// Unload orphan dl packages 
+	Package::UnLoadReleasedDynamicallyLoadedPackages();
+	
+
+	
+	
+      }
+    
+    bbtkDebugMessage("package",1,"<== Package::Release('"<<
+		     packname<<"')"<<bbtkendl);
+    if (!pack.expired())
+      {
+	long dyn = pack.lock()->mDynamicLibraryHandler ? 1:0; 
+	long ndesc = pack.lock()->GetBlackBoxMap().size();
+	long nrefs = pack.use_count();
+	
+	bbtkDebugMessage("package",1," ... Package still alive ("
+			 <<nrefs<<" refs / "
+			 <<ndesc<<" descr / dyn="
+			 <<dyn<<")"<<std::endl);
+      }
+    else 
+      {
+	bbtkDebugMessage("package",1,"   ... Package has been released"
+			 <<std::endl);
+      }
+  }
+  //==========================================================================
+
+  //==========================================================================
+  /// "Releases" the package
+  /// Signals the package that it can free the given descriptor
+  /// if they are no more used and free itself if it is no
+  /// more used
+  /// Note : Any non-weak pointer on the package must have been freed
+  void Package::ReleaseBlackBoxDescriptor(Package::WeakPointer pack,
+					  BlackBoxDescriptor::WeakPointer descr)
+  {
+    std::string packname = pack.lock()->mName;
+    std::string dname = descr.lock()->GetTypeName();    
+    bbtkDebugMessage("package",3,"==> Package::ReleaseBlackBoxDescriptor('"<<
+		     packname<<"','"<<dname<<"')"<<bbtkendl);
+
+    long dyn = pack.lock()->mDynamicLibraryHandler ? 1:0; 
+    long ndesc = pack.lock()->GetBlackBoxMap().size();
+    long nrefs = pack.use_count();
+
+    bbtkDebugMessage("package",3," "<<nrefs<<" refs / "
+		     <<ndesc<<" descr / dynamically loaded = "
+		     <<dyn<<std::endl);
+
+    // A package is "free" from any external reference iff :
+    // i) It is not dynamically loaded and nrefs == ndesc 
+    // (each desc references its package) or
+    // ii) It is dynamically loaded and nrefs == ndesc + 1
+    // (A dynamic library holds a static pointer on the package it contains
+    //  which is allocated when the PACKAGENAMEGetPackage() func is called,
+    //  and descallocated (reset) by PACKAGENAMEDeletePackage())
+    if (nrefs == ndesc + dyn) 
+      {
+	PackageReleaseBlackBoxDescriptorInternal(pack,dname);
+      }
+    
+    // If the package is released and dynamically loaded 
+    // then put it in the static list mReleasedDynamicallyLoadedPackages
+    UnLoadDynamicLibrary(pack,false);
+        
+    bbtkDebugMessage("package",3,"<== Package::ReleaseBlackBoxDescriptor('"<<
+		     packname<<"','"<<dname<<"')"<<bbtkendl);
     /*
-    AdaptorMapType::const_iterator j;
-    for (j=mAdaptorMap.begin();
-         j!=mAdaptorMap.end();
-       ++j) 
+    if (!pack.expired())
       {
-         delete j->second;
+	long dyn = pack.lock()->mDynamicLibraryHandler ? 1:0; 
+	long ndesc = pack.lock()->GetBlackBoxMap().size();
+	long nrefs = pack.use_count();
+	
+	bbtkDebugMessage("package",3," ... Package still alive ("
+			 <<nrefs<<" refs / "
+			 <<ndesc<<" descr / dyn="
+			 <<dyn<<")"<<std::endl);
       }
-    */ 
-    bbtkDebugDecTab("Kernel",7);
+    else 
+      {
+	bbtkDebugMessage("package",3,"   ... Package has been released"
+			 <<std::endl);
+      }  
+    */
+  }
+  //==========================================================================    
+
+  //==========================================================================
+  /// Opens a dynamic library which contains a bbtk package
+  /// Returns the handler 
+  /// Load the package management symbols from the lib
+  /// returns false if a problem occured hence can be used 
+  /// to test that a dyn lib is a valid bbtk package lib
+  /// NB : The BBTK version exported from the library 
+  ///      is tested against the current bbtk version
+  DynamicLibraryHandler Package::OpenDynamicLibrary
+  ( const std::string& libname,
+    const std::string& package_name,
+    DLGetPackageFunction& getpack,
+    DLDeletePackageFunction& delpack)
+  {
+    bbtkDebugMessage("package",3,"==> Package::OpenDynamicLibrary("
+		     <<libname<<")"<<std::endl);
+#if defined(__GNUC__)
+
+    // Open shared lib
+    void *handler;
+    handler = dlopen(libname.c_str(),
+		     BBTK_RTLD_TIME | BBTK_RTLD_SCOPE );
+    if (!handler)
+      {
+	bbtkMessage("package",2,
+		    "Could not open shared library [" <<libname<<"] : "
+		    <<dlerror() << std::endl);
+	return 0;
+      }
+
+    bbtkDebugMessage("package",3,"* Shared lib ["<<libname<<"] open"<<std::endl);
+
+    // Loads the Package bbtk version function 
+    std::string getvername(package_name);
+    getvername += 
+      BBTK_STRINGIFY_SYMBOL(BBTK_GET_PACKAGE_BBTK_VERSION_FUNCTION_NAME);
+    DLGetPackageBBTKVersionFunction getbbtkversion 
+      = (DLGetPackageBBTKVersionFunction)(dlsym(handler,getvername.c_str()));
+    if (!getbbtkversion)
+      {
+	bbtkDebugMessage("package",3,"***"<<std::endl);
+	bbtkMessage("package",2,
+		    "Shared library ["<<libname
+		    <<"] is not a valid bbtk package."
+		    <<" Symbol ["<<getvername<<"] :"<<dlerror()<< std::endl);
+ 	dlclose(handler);
+	return 0;
+      }
+
+    bbtkDebugMessage("package",3,"* Symbol ["<<getvername
+		     <<"] found"<<std::endl);
+    // version matches ?
+    if (getbbtkversion() != bbtk::GetVersion())
+      {
+	bbtkMessage("package",2,
+		    "Shared library ["<<libname
+		    <<"] was build with bbtk version "
+		    <<getbbtkversion()
+		    <<" but the current program runs with version "
+		    <<bbtk::GetVersion()<<" : cannot load it"<<std::endl);
+	dlclose(handler);
+	return 0;
+	
+      }
+
+    bbtkDebugMessage("package",3,"* Package bbtk version '"<<getbbtkversion()<<"' matches"<<std::endl);
+	    // Loads the Package get function
+    std::string getpackname(package_name);
+    getpackname += BBTK_STRINGIFY_SYMBOL(BBTK_GET_PACKAGE_FUNCTION_NAME);
+    getpack = (DLGetPackageFunction)(dlsym(handler, getpackname.c_str()));
+    if (!getpack)
+      {
+	bbtkMessage("package",2,
+		    "Shared library ["<<libname
+		    <<"] is not a valid bbtk package."
+		    <<" Symbol ["<<getpackname<<"] :"<<dlerror()<< std::endl);
+	dlclose(handler);
+	return 0;
+      }
+  
+    bbtkDebugMessage("package",3,"* Symbol ["<<getpackname<<"] found"<<std::endl);
+    // Loads the Package delete function
+					  
+    std::string delpackname(package_name);
+    delpackname += BBTK_STRINGIFY_SYMBOL(BBTK_DEL_PACKAGE_FUNCTION_NAME);
+    delpack = (DLDeletePackageFunction)(dlsym(handler, delpackname.c_str()));
+    if (!delpack)
+      {
+	bbtkMessage("package",2,
+		    "Shared library ["<<libname
+		    <<"] is not a valid bbtk package."
+		    <<" Symbol ["<<delpackname<<"] :"<<dlerror()<< std::endl);
+	dlclose(handler);
+	return 0;
+      }
+    bbtkDebugMessage("package",3,"* Symbol ["<<delpackname<<"] found"<<std::endl);		  
+#elif defined(_WIN32)
+    
+    HINSTANCE handler;
+    
+    SetErrorMode(0);
+    // Open shared lib
+    handler = LoadLibrary(libname.c_str());
+    if (!handler)
+      {
+	bbtkMessage("package",2,
+		    "Could not open shared library [" <<libname<<"]"
+		    << std::endl);
+	return 0;
+      }
+    
+    // Loads the Package bbtk version function 
+    std::string getvername(package_name);
+    getvername += 
+      BBTK_STRINGIFY_SYMBOL(BBTK_GET_PACKAGE_BBTK_VERSION_FUNCTION_NAME);
+    DLGetPackageBBTKVersionFunction getbbtkversion 
+      = (DLGetPackageBBTKVersionFunction)(GetProcAdress(handler,
+							getvername.c_str()));
+    if (!getbbtkversion)
+      {
+	FreeLibrary(handler);
+	bbtkMessage("package",2,
+		    "Shared library ["<<libname
+		    <<"] is not a valid bbtk package."
+		    <<" Symbol ["<<getbbtkversion<<"] not found"<< std::endl);
+	return 0;
+      }
+    
+    // version matches ?
+    if (getbbtkversion() != bbtk::GetVersion())
+      {
+	FreeLibrary(handler);
+	bbtkMessage("package",2,
+		    "Shared library ["<<libname
+		    <<"] was build with bbtk version "
+		    <<getbbtkversion()
+		    <<" but the current program runs with version "
+		    <<bbtk::GetVersion()<<" : cannot load it"<<std::endl);
+	return 0;
+	
+      }
+
+     // Loads the Package get function
+    std::string getpackname(package_name);
+    getpackname += BBTK_STRINGIFY_SYMBOL(BBTK_GET_PACKAGE_FUNCTION_NAME);
+    getpack = (DLGetPackageFunction)(GetProcAdress(handler, getpackname.c_str()));
+    if (!getpack)
+      {
+	FreeLibrary(handler);
+	bbtkMessage("package",2,
+		    "Shared library ["<<libname
+		    <<"] is not a valid bbtk package."
+		    <<" Symbol ["<<getpackname<<"] not found"<< std::endl);
+	return 0;
+      }
+    
+    // Loads the Package delete function
+    std::string delpackname(package_name);
+    delpackname += BBTK_STRINGIFY_SYMBOL(BBTK_DEL_PACKAGE_FUNCTION_NAME);
+    delpack = (DLDeletePackageFunction)(GetProcAdress(handler, delpackname.c_str()));
+    if (!delpack)
+      {
+	FreeLibrary(handler);
+	bbtkMessage("package",2,
+		    "Shared library ["<<libname
+		    <<"] is not a valid bbtk package."
+		    <<" Symbol ["<<delpackname<<"] not found"<< std::endl);
+	return 0;
+      }
+					 
+#else
+    bbtkError("neither __GNUC__ nor _WIN32 ?!? How did you compile ?");
+#endif
+    
+    return handler;
+  }
+  //==========================================================================
+  
+  //==========================================================================
+  /// Loads a package from a dynamic library
+  Package::Pointer Package::CreateFromDynamicLibrary(const std::string& libname,
+						     const std::string& pkgname,
+						     const std::string& path)
+  {
+    bbtkDebugMessage("package",1,"==> Package::CreateFromDynamicLibrary("
+		     <<libname<<")"<<std::endl);
+
+    DLGetPackageFunction gf;
+    DLDeletePackageFunction df;
+    DynamicLibraryHandler h = Package::OpenDynamicLibrary(libname,
+							  pkgname,
+							  gf,df);
+    if (h==0) return Package::Pointer(); 
+    Package::Pointer p = gf();
+    p->mDynamicLibraryHandler = h;
+    p->mDLDeletePackageFunction = df;
+    
+    std::string separator =
+      ConfigurationFile::GetInstance().Get_file_separator ();
+    //BBTK_STRINGIFY_SYMBOL(BBTK_DOC_REL_PATH)
+    std::string docreldoc = 
+      separator + "bbdoc" + separator + pkgname + separator + "index.html";
+    std::string reldoc = 
+      ".." + separator + ".." + docreldoc;
+    std::string doc = path + separator + ".." + separator
+      + BBTK_STRINGIFY_SYMBOL(BBTK_DOC_REL_PATH)
+      + docreldoc;
+    
+    p->SetDocURL(doc);
+    p->SetDocRelativeURL(reldoc);
+    
+    bbtkDebugMessage("package",1,"<== Package::CreateFromDynamicLibrary("
+		     <<libname<<") .. OK"<<std::endl);
+    return p;
+  }
+  //==========================================================================
+
+
+
+
+  //==========================================================================
+  /// UnLoads the package dynamic library (if any)
+  void Package::UnLoadDynamicLibrary(Package::WeakPointer pack, bool doit)
+  {
+    if (pack.expired() || (!pack.lock()->mDynamicLibraryHandler))
+      return;
+    
+    
+    std::string packname = pack.lock()->GetName();
+    bbtkDebugMessage("package",3,"==> Package::UnLoadDynamicLibrary('"
+		     <<packname<<"')"
+		     <<std::endl);
+    
+    if (!pack.lock()->GetBlackBoxMap().empty())
+      {
+	
+	bbtkDebugMessage("package",3,"   Package not empty ... abort"
+			 <<std::endl);
+	return;
+	/*
+	bbtkGlobalError("Package::UnLoadDynamicLibrary('"<<packname<<") : "
+			<<"BlackBoxMap not empty "
+			<<BBTK_INTERNAL_ERROR_MESSAGE);
+	*/
+	
+      }
+
+    if (pack.use_count()!=1)
+      {
+	bbtkGlobalError("Package::UnLoadDynamicLibrary('"<<packname<<") : "
+			<<"empty dl package with external refs"
+			<<BBTK_INTERNAL_ERROR_MESSAGE);
+      } 
+
+    if (doit) 
+      {
+	UnLoad(pack);
+	bbtkDebugMessage("package",3,"   ... dynamic library closed"
+			 <<std::endl);    
+      }
+    else 
+      {
+	mReleasedDynamicallyLoadedPackages.insert(pack);
+	bbtkDebugMessage("package",3,
+			 "   ... package put in the 'to unload' list"
+			 <<std::endl);
+
+      }
+
+    bbtkDebugMessage("package",3,"<== Package::UnLoadDynamicLibrary('"
+		     <<packname<<"')"
+		     <<std::endl);
+    
+  }
+  //==========================================================================
+
+  //==========================================================================
+  /// UnLoads released packages that were loaded dynamically
+  /// see UnLoadDynamicLibrary and ReleaseBlackBoxDescriptor
+  void Package::UnLoadReleasedDynamicallyLoadedPackages()
+  {
+    bbtkDebugMessage("package",2,"==> Package::UnLoadReleasedDynamicallyLoadedPackages()"<<std::endl);
+
+    std::set<Package::WeakPointer>::iterator i;
+    for (i=mReleasedDynamicallyLoadedPackages.begin();
+	 i!=mReleasedDynamicallyLoadedPackages.end();
+	 ++i)
+      {
+	if (!i->expired()) UnLoad(*i);
+      }
+    bbtkDebugMessage("package",2,"<== Package::UnLoadReleasedDynamicallyLoadedPackages()"<<std::endl);
   }
   //==========================================================================
 
+  //==========================================================================
+  void Package::UnLoad(Package::WeakPointer pack)
+  {
+    std::string packname = pack.lock()->GetName();
+    bbtkDebugMessage("package",2,"==> Package::UnLoad("<<packname<<")"<<std::endl);
 
+    Package* p = pack.lock().get();
+    
+    DynamicLibraryHandler h = p->mDynamicLibraryHandler;
+    
+    // deletes the package
+    p->mDLDeletePackageFunction();
+    
+    // closes the dl handler
+#if defined(__GNUC__)  
+    dlclose(h);  
+#elif defined(_WIN32)
+    FreeLibrary(h);
+#endif
+
+    bbtkDebugMessage("package",2,"   ... dynamic library unloaded"<<std::endl);
+  }
+  //==========================================================================
 
   //==========================================================================
   /// Creates an instance of a black box of type <type> with name <name>
-  BlackBox* Package::NewBlackBox(const std::string& type, 
-				    const std::string& name) const
+  BlackBox::Pointer Package::NewBlackBox(const std::string& type, 
+					 const std::string& name) const
   {
     bbtkDebugMessageInc("Kernel",8,"Package<"<<GetName()<<">::NewBlackBox(\""<<type<<"\",\""<<name<<"\")"<<bbtkendl);
     
@@ -107,9 +643,9 @@ namespace bbtk
     if (i == mBlackBoxMap.end())  
     {
 	   bbtkDebugDecTab("Kernel",8);
-	   return 0;
+	   return BlackBox::Pointer();
     }
-    BlackBox* bb =i->second->CreateInstance(name);
+    BlackBox::Pointer bb =i->second->NewBlackBox(name);
     bbtkDebugDecTab("Kernel",8);
     return bb;   
 
@@ -121,7 +657,7 @@ namespace bbtk
   //==========================================================================
   /// Creates an instance of an adaptor of input type <typein> and 
   /// output type <typeout>  with name <name>
-  BlackBox* Package::NewAdaptor(const DataInfo& typein,
+  BlackBox::Pointer Package::NewAdaptor(const DataInfo& typein,
 				const DataInfo& typeout,
 				const std::string& name) const
   {
@@ -137,9 +673,9 @@ namespace bbtk
     if (i == mAdaptorMap.end())  
       {
 	bbtkDebugDecTab("Kernel",8);
-	return 0;
+	return BlackBox::Pointer();
       }
-    BlackBox* bb =i->second->CreateInstance(name);
+    BlackBox::Pointer bb =i->second.lock()->NewBlackBox(name);
     bbtkDebugDecTab("Kernel",8);
     return bb;   
 
@@ -149,7 +685,7 @@ namespace bbtk
   //==========================================================================
   /// Creates an instance of an adaptor of input type <typein> and 
   /// output type <typeout>  with name <name>
-  BlackBox* Package::NewWidgetAdaptor(const DataInfo& typein,
+  BlackBox::Pointer Package::NewWidgetAdaptor(const DataInfo& typein,
 				      const DataInfo& typeout,
 				      const std::string& name) const
   {
@@ -165,9 +701,9 @@ namespace bbtk
     if (i == mAdaptorMap.end())  
       {
 	bbtkDebugDecTab("Kernel",8);
-	return 0;
+	return BlackBox::Pointer();
       }
-    BlackBox* bb =i->second->CreateInstance(name);
+    BlackBox::Pointer bb =i->second.lock()->NewBlackBox(name);
     bbtkDebugDecTab("Kernel",8);
     return bb;   
 
@@ -190,7 +726,9 @@ namespace bbtk
 			<<typein<<","
 			<<typeout<<")"<<bbtkendl);
    
-    AdaptorKey key(typein,typeout,
+    AdaptorKey key(/*typein*/
+		   DataInfo(typeid(void),""),
+		   typeout,
 		   BlackBoxDescriptor::DEFAULT_WIDGET_ADAPTOR);
     // First try to find a single widget adaptor
     AdaptorMapType::const_iterator i = mAdaptorMap.find(key);
@@ -199,7 +737,7 @@ namespace bbtk
 	bbtkDebugDecTab("Kernel",8);
 	return false;
       }
-    adaptor = i->second->GetTypeName();
+    adaptor = i->second.lock()->GetTypeName();
     bbtkDebugDecTab("Kernel",8);
     return true;   
 
@@ -230,7 +768,7 @@ namespace bbtk
 	bbtkDebugDecTab("Kernel",8);
 	return false;
       }
-    adaptor = i->second->GetTypeName();
+    adaptor = i->second.lock()->GetTypeName();
     bbtkDebugDecTab("Kernel",8);
     return true;   
 
@@ -240,7 +778,7 @@ namespace bbtk
 
   //==========================================================================
   /// Registers a black box descriptor in the package
-  bool Package::RegisterBlackBox(BlackBoxDescriptor* d) 
+  bool Package::RegisterBlackBox(BlackBoxDescriptor::Pointer d) 
   {
     bbtkDebugMessageInc("Kernel",8,"Package<"<<GetName()<<">::RegisterBlackBox(\""<<d->GetTypeName()<<"\")"<<std::endl);
     
@@ -253,24 +791,22 @@ namespace bbtk
       }
 
     mBlackBoxMap[d->GetTypeName()] = d;
-    d->Reference();
-    d->SetPackage(this);
+    //    d->Reference();
+    d->SetPackage(GetThisPointer<Package>());
     
     // If it is a default adaptor, also register it in the adaptors map
-    if ( (d->GetKind() == BlackBoxDescriptor::DEFAULT_ADAPTOR) ||
-	 (d->GetKind() == BlackBoxDescriptor::DEFAULT_WIDGET_ADAPTOR) )
+    if ( d->GetKind() == BlackBoxDescriptor::DEFAULT_ADAPTOR )
       {
 	bbtkDebugMessage("Kernel",8,"Package<"<<GetName()<<">::RegisterBlackBox(\""<<d->GetTypeName()<<"\") : The box is an adaptor, inserting it in adaptors map ..."<<std::endl);   
+	
 	TypeInfo typein = d->GetInputDescriptor("In")->GetTypeInfo();
 	TypeInfo typeout = d->GetOutputDescriptor("Out")->GetTypeInfo();
 	DataInfo infoin(typein,d->GetInputDescriptor("In")->GetNature());
 	DataInfo infoout(typeout,d->GetOutputDescriptor("Out")->GetNature());
-	bbtkDebugMessage("Kernel",9,"Adaptor "<<infoin<<" to "<<infoout 
-			 <<" - kind="<<d->GetKind()<<std::endl);  
-
 	AdaptorKey key(infoin,infoout,d->GetKind());
-
-	AdaptorMapType::const_iterator i = mAdaptorMap.find(key);
+	
+	AdaptorMapType::const_iterator i;
+	i = mAdaptorMap.find(key);	  
 	if (i == mAdaptorMap.end())  
 	  {
 	    mAdaptorMap[key] = d;
@@ -278,17 +814,47 @@ namespace bbtk
 	// If already an adaptor registered : error
 	else 
 	  {
-	    if (i->second->GetTypeName() != d->GetTypeName()) 
+	    if (i->second.lock()->GetTypeName() != d->GetTypeName()) 
 	      {
 		bbtkError("Package <"<<GetName()<<
 			  "> : trying to register black box <"
 			  <<d->GetTypeName()
 			  <<"> as default adaptor but there is already a default adaptor registered (<"
-			  <<i->second->GetTypeName()<<">)");
+			  <<i->second.lock()->GetTypeName()<<">)");
+	      }
+	  }
+      }
+    // If it is a default adaptor, also register it in the adaptors map
+    else if ( d->GetKind() == BlackBoxDescriptor::DEFAULT_WIDGET_ADAPTOR)
+      {
+	bbtkDebugMessage("Kernel",8,"Package<"<<GetName()<<">::RegisterBlackBox(\""<<d->GetTypeName()<<"\") : The box is a widget adaptor, inserting it in adaptors map ..."<<std::endl);   
+	
+	TypeInfo typeout = d->GetOutputDescriptor("Out")->GetTypeInfo();
+	DataInfo infoin(typeid(void),"");
+	DataInfo infoout(typeout,d->GetOutputDescriptor("Out")->GetNature());
+	AdaptorKey key(infoin,infoout,d->GetKind());
+
+	AdaptorMapType::const_iterator i;
+	i = mAdaptorMap.find(key);	  
+	if (i == mAdaptorMap.end())  
+	  {
+	    mAdaptorMap[key] = d;
+	  }
+	// If already an adaptor registered : error
+	else 
+	  {
+	    if (i->second.lock()->GetTypeName() != d->GetTypeName()) 
+	      {
+		bbtkError("Package <"<<GetName()<<
+			  "> : trying to register black box <"
+			  <<d->GetTypeName()
+			  <<"> as default widget adaptor but there is already a default adaptor registered (<"
+			  <<i->second.lock()->GetTypeName()<<">)");
 	      }
 	  }
       }
     
+
     bbtkDebugDecTab("Kernel",8);
    
     return true;
@@ -312,34 +878,6 @@ namespace bbtk
   }
   //===================================================================
 
-  //==========================================================================
-  /// UnRegisters a black box descriptor from the package
-  void Package::UnRegisterBlackBox(const std::string& name) 
-  {
-    bbtkDebugMessageInc("Kernel",8,"Package<"<<GetName()<<">::UnRegisterBlackBox(\""<<name<<"\")"<<std::endl);
-    // Looking into the bb map
-    BlackBoxMapType::iterator i = mBlackBoxMap.find(name);
-    if (i == mBlackBoxMap.end())  
-    {
-       bbtkDebugDecTab("Kernel",8);
-       bbtkError("UnRegister : The package <"<<GetName()<<"> does not contains the black box <"<<name<<">");
-     }
-    mBlackBoxMap.erase(i);
-
-    // Is it also in the adaptors map ?
-    /*
-    AdaptorMapType::iterator j = mAdaptorMap.find(name);
-    if (j != mAdaptorMap.end())  
-      {
-         mAdaptorMap.erase(j);
-      }
-    */    
-
-    i->second->UnReference();
-
-    bbtkDebugDecTab("Kernel",8);    
-  }
-  //==========================================================================
 
   //==========================================================================
   /// Changes the name of a black box type
@@ -362,23 +900,6 @@ namespace bbtk
   }
   //==========================================================================
 
-  /*
-
-  //==========================================================================
-  /// Registers an adaptor descriptor in the package
-  bool Package::RegisterAdaptor(BlackBoxDescriptor* d) 
-  {
-    bbtkDebugMessage("Kernel",8,"Package<"<<GetName()<<">::RegisterAdaptor(\""<<d->GetTypeName()<<"\")"<<std::endl);
-    
-    TypeInfo typein = d->GetInputDescriptor("In")->GetTypeInfo();
-    TypeInfo typeout = d->GetOutputDescriptor("Out")->GetTypeInfo();
-    AdaptorKey key(typein,typeout);
-    
-    mAdaptorMap[key] = d;
-    return true;
-  }
-  //==========================================================================
-  */
 
 
   //==========================================================================
@@ -641,7 +1162,7 @@ namespace bbtk
   	s << "<TR><TD style='vertical-align: top;'><b> Version </b></TD><TD style='vertical-align: top;'> : </TD><TD style='vertical-align: top;'> " 
 	  << GetVersion() << "</TD></TR>\n";
 	s << "<TR><TD style='vertical-align: top;'><b> bbtk Version </b></TD><TD style='vertical-align: top;'> : </TD><TD style='vertical-align: top;'> " 
-	  << GetBBTKVersion() << "</TD></TR>\n";
+	  << bbtk::GetVersion() << "</TD></TR>\n";
 	s << "</TABLE>\n";
       }
 
@@ -770,5 +1291,61 @@ namespace bbtk
     bbtkDebugDecTab("Kernel",9);
   }
   //==========================================================================
+  
+  //==========================================================================
+  std::string  Package::GetObjectName() const 
+  { 
+    return std::string("Package '")+mName+std::string("'"); 
+  }
+  //==========================================================================
+
+  //==========================================================================
+  std::string Package::GetObjectInfo() const 
+  {
+    std::stringstream i;
+    i << "  - "<<mBlackBoxMap.size() << " boxes" << std::endl;
+    if (mDynamicLibraryHandler) 
+      {
+	i<< "  - Loaded from dynamic library"<<std::endl;
+      }
+    return i.str();
+  }
+  //==========================================================================
+
+
+  //==========================================================================
+  size_t  Package::GetObjectSize() const 
+  {
+    size_t s = Superclass::GetObjectSize();
+    s += Package::GetObjectInternalSize();
+    return s;
+  }
+  //==========================================================================
+  //==========================================================================
+  size_t  Package::GetObjectInternalSize() const 
+  {
+    size_t s = sizeof(Package);
+    return s;
+  }
+  //==========================================================================
+  //==========================================================================
+  size_t  Package::GetObjectRecursiveSize() const 
+  {
+    size_t s = Superclass::GetObjectRecursiveSize();
+    s += Package::GetObjectInternalSize();
+    
+    BlackBoxMapType::const_iterator i;
+    for (i = mBlackBoxMap.begin(); i!=mBlackBoxMap.end(); ++i )
+      {
+	s += i->second->GetObjectRecursiveSize();
+      }
+    return s;
+  }
+  //==========================================================================
+  
+  //==========================================================================
+  std::set<Package::WeakPointer> 
+  Package::mReleasedDynamicallyLoadedPackages;
+  //==========================================================================
 }