Re: [luabind] Solved. Re: Fwd: Inspecting objects

[Daniel W. <da...@bo...>]

On Mon, Mar 22, 2010 at 09:17:03AM -0700, Thomas Nelson wrote:
> On 3/22/2010 2:36 AM, Daniel Wallin wrote:
> > On Fri, Mar 19, 2010 at 11:11:18AM -0700, Thomas Nelson wrote:
> >
> >> Solved... My bad.  It turns out it had to do with how I was exporting
> >> the object... turns out it's helpful to wait till after you've created
> >> the instance before you inject a back ref.
> >>
> >> Any interest on the list for a patch with the following enhancements:
> >> 1) Export a luabind::wrap_base derived object to an arbitrary
> >> table/namespace in the lua from C++
> >>
> > What does "export" mean in this context?
> >
> Certainly, at the moment it behaves very similarly to luabind::globals 
> function and in fact it may be a good idea to ultimately put the 
> functionality there.  It is actually an interim step on the way allow me 
> to make a class created in C++ visible in the lua as a specified lua 
> subclass (I need that functionality for scene loading, but it is gonna 
> be a week or two before I get around to actually writing the code to do 
> this.)

OK, I still don't understand what "export" means here, but I'll wait for
the unified diff and look at the code.

> >> 2) Allow methods to be overridden on a per instance basis
> >>
> > This can be done with the code on the master branch, see:
> >
> >    http://github.com/luabind/luabind/commit/4b52671
> >
> And that's what I get from working from the release.  How stable is the 
> trunk at the moment?  I'd rather use your code for this than mine.

Well, it doesn't have the latest changes from the 0.9 branch, because
the v0.9.1 release hasn't happened yet, but it should be stable. In any
case, you should be able to cherry-pick that particular change.

> > I couldn't apply the patch you sent later in this thread. Could you turn
> > it into a unified diff (diff -u)?
> >
> Of course, It will be a couple days before I get it together though.

OK, thanks.

-- 
Daniel Wallin
BoostPro Computing
http://www.boostpro.com