From 5c8f0cc966576c707ce73ca54d815b2cf56bba0a Mon Sep 17 00:00:00 2001
From: Thomas Geymayer <tomgey@gmail.com>
Date: Sun, 2 Jun 2013 21:20:47 +0200
Subject: [PATCH] Make nasal::Ghost usable with weak_ptr

---
 simgear/nasal/cppbind/Ghost.hxx        | 14 ++++++++++----
 simgear/nasal/cppbind/cppbind_test.cxx |  5 +++++
 2 files changed, 15 insertions(+), 4 deletions(-)

diff --git a/simgear/nasal/cppbind/Ghost.hxx b/simgear/nasal/cppbind/Ghost.hxx
index 81165784..c403a307 100644
--- a/simgear/nasal/cppbind/Ghost.hxx
+++ b/simgear/nasal/cppbind/Ghost.hxx
@@ -35,6 +35,12 @@
 
 #include <map>
 
+template<class T>
+inline T* get_pointer(boost::weak_ptr<T> const& p)
+{
+  return p.lock().get();
+}
+
 /**
  * Bindings between C++ and the Nasal scripting language
  */
@@ -569,7 +575,7 @@ namespace nasal
                                     ++parent )
           {
             pointer ptr = fromNasal(c, *parent);
-            if( ptr.get() )
+            if( get_pointer(ptr) )
               return ptr;
           }
         }
@@ -592,7 +598,7 @@ namespace nasal
        */
       static pointer* createInstance(const pointer& ptr)
       {
-        return ptr.get() ? new pointer(ptr) : 0;
+        return get_pointer(ptr) ? new pointer(ptr) : 0;
       }
 
       static pointer getPtr(void* ptr)
@@ -606,14 +612,14 @@ namespace nasal
       static raw_type* getRawPtr(void* ptr)
       {
         if( ptr )
-          return static_cast<pointer*>(ptr)->get();
+          return get_pointer(*static_cast<pointer*>(ptr));
         else
           return 0;
       }
 
       static raw_type* getRawPtr(const pointer& ptr)
       {
-        return ptr.get();
+        return get_pointer(ptr);
       }
 
       void addDerived( const internal::GhostMetadata* derived_meta,
diff --git a/simgear/nasal/cppbind/cppbind_test.cxx b/simgear/nasal/cppbind/cppbind_test.cxx
index ed5cb400..29cb556a 100644
--- a/simgear/nasal/cppbind/cppbind_test.cxx
+++ b/simgear/nasal/cppbind/cppbind_test.cxx
@@ -5,6 +5,7 @@
 #include "NasalString.hxx"
 
 #include <boost/shared_ptr.hpp>
+#include <boost/weak_ptr.hpp>
 
 #include <cstring>
 #include <iostream>
@@ -66,6 +67,8 @@ typedef boost::shared_ptr<Derived> DerivedPtr;
 typedef boost::shared_ptr<DoubleDerived> DoubleDerivedPtr;
 typedef boost::shared_ptr<DoubleDerived2> DoubleDerived2Ptr;
 
+typedef boost::weak_ptr<Derived> DerivedWeakPtr;
+
 naRef derivedFreeMember(Derived&, const nasal::CallContext&) { return naNil(); }
 naRef f_derivedGetX(naContext c, const Derived& d)
 {
@@ -165,6 +168,8 @@ int main(int argc, char* argv[])
     .member("base", &DoubleDerived2::getBase)
     .method("doIt", &DoubleDerived2::doSomeBaseWork);
 
+  Ghost<DerivedWeakPtr>::init("DerivedWeakPtr");
+
   VERIFY( Ghost<BasePtr>::isInit() );
   nasal::to_nasal(c, DoubleDerived2Ptr());
 
-- 
2.39.5