X-Git-Url: http://review.tizen.org/git/?a=blobdiff_plain;f=zypp%2FZYppFactory.cc;h=45fc1c28bd7f9d41bf174ed680abb72a9890ca77;hb=f2f36ebec8b34e01f91f190cbd0b9f4d3c2c4510;hp=03ea8973b450d94b6f2c1861fed894e5ec7951f1;hpb=d492bc6ff98c3a2917afcaad406ef9586a9ba8f5;p=platform%2Fupstream%2Flibzypp.git

diff --git a/zypp/ZYppFactory.cc b/zypp/ZYppFactory.cc
index 03ea897..45fc1c2 100644
--- a/zypp/ZYppFactory.cc
+++ b/zypp/ZYppFactory.cc
@@ -282,35 +282,14 @@ namespace zypp
 
   };
 
-  ///////////////////////////////////////////////////////////////////
   namespace
   {
-    static weak_ptr<ZYpp>		_theZYppInstance;
-    static scoped_ptr<ZYppGlobalLock>	_theGlobalLock;		// on/off in sync with _theZYppInstance
-
-    ZYppGlobalLock & globalLock()
+    static ZYppGlobalLock & globalLock()
     {
-      if ( !_theGlobalLock )
-	_theGlobalLock.reset( new ZYppGlobalLock );
-      return *_theGlobalLock;
+      static ZYppGlobalLock lock;
+      return lock;
     }
-  } //namespace
-  ///////////////////////////////////////////////////////////////////
-
-  ///////////////////////////////////////////////////////////////////
-  //
-  //	CLASS NAME : ZYpp
-  //
-  ///////////////////////////////////////////////////////////////////
-
-  ZYpp::ZYpp( const Impl_Ptr & impl_r )
-  : _pimpl( impl_r )
-  {
-  }
-
-  ZYpp::~ZYpp()
-  {
-    _theGlobalLock.reset();
+    bool           _haveZYpp = false;
   }
 
   ///////////////////////////////////////////////////////////////////
@@ -334,12 +313,31 @@ namespace zypp
   //
   ///////////////////////////////////////////////////////////////////
 
+  ///////////////////////////////////////////////////////////////////
+  //
+  //	METHOD NAME : ZYppFactory::instance
+  //	METHOD TYPE : ZYppFactory
+  //
   ZYppFactory ZYppFactory::instance()
-  { return ZYppFactory(); }
+  {
+    return ZYppFactory();
+  }
 
+  ///////////////////////////////////////////////////////////////////
+  //
+  //	METHOD NAME : ZYppFactory::ZYppFactory
+  //	METHOD TYPE : Ctor
+  //
   ZYppFactory::ZYppFactory()
-  {}
+  {
 
+  }
+
+  ///////////////////////////////////////////////////////////////////
+  //
+  //	METHOD NAME : ZYppFactory::~ZYppFactory
+  //	METHOD TYPE : Dtor
+  //
   ZYppFactory::~ZYppFactory()
   {}
 
@@ -347,7 +345,8 @@ namespace zypp
   //
   ZYpp::Ptr ZYppFactory::getZYpp() const
   {
-    ZYpp::Ptr _instance = _theZYppInstance.lock();
+    static ZYpp::Ptr _instance;
+
     if ( ! _instance )
     {
       if ( geteuid() != 0 )
@@ -388,6 +387,7 @@ namespace zypp
 	      }
 	    }
 	  }
+
 	}
 	if ( failed )
 	{
@@ -400,11 +400,9 @@ namespace zypp
 	}
       }
       // Here we go...
-      static ZYpp::Impl_Ptr _theImplInstance;	// for now created once
-      if ( !_theImplInstance )
-	_theImplInstance.reset( new ZYpp::Impl );
-      _instance.reset( new ZYpp( _theImplInstance ) );
-      _theZYppInstance = _instance;
+      _instance = new ZYpp( ZYpp::Impl_Ptr(new ZYpp::Impl) );
+      if ( _instance )
+        _haveZYpp = true;
     }
 
     return _instance;
@@ -413,7 +411,7 @@ namespace zypp
   ///////////////////////////////////////////////////////////////////
   //
   bool ZYppFactory::haveZYpp() const
-  { return !_theZYppInstance.expired(); }
+  { return _haveZYpp; }
 
   /******************************************************************
   **