On Tue, Oct 20, 2009 at 8:36 AM, Troy A. Griffitts <scr...@crosswire.org>wrote:
> Thanks for the patch Nathan, > > Is this really necessary? This has been in the code for quite some time > and BPBible has been working with swig bindings for quite some time with > this in the code. I don't believe this inner class should ever be > exposed in the bindings, as it is a mechanism we use in the > implementation of a specific derived class, and not a basic interface. > > This isn't necessary for me; SWIG will give warnings about nested classes, but not errors (they just won't be wrapped). It is actually possible to get round these restrictions, but here it isn't worth it. The place I'd actually be a little more worried about the nested classes is in the av11n code, with VerseMgr::System, etc. God Bless, Ben ------------------------------------------------------------------------------------------- Multitudes, multitudes, in the valley of decision! For the day of the LORD is near in the valley of decision. Giôên 3:14 (ESV)
_______________________________________________ sword-devel mailing list: sword-devel@crosswire.org http://www.crosswire.org/mailman/listinfo/sword-devel Instructions to unsubscribe/change your settings at above page