Re: [PATCH] OpenSolaris/ZFS: C++ compatibility

2011-02-08 Thread Baptiste Daroussin
> I've developed similar API during HAST work, maybe it is a good starting > point? src/sbin/hastd/nv.{c,h}. > What about pushing this into some public libraries, I think this could be usefull in libc ou libutil or something like that? regards, Bapt ___

Re: [PATCH] OpenSolaris/ZFS: C++ compatibility

2011-02-05 Thread Garrett Cooper
On Sat, Feb 5, 2011 at 7:29 PM, Justin T. Gibbs wrote: > On 2/5/2011 3:06 PM, Pawel Jakub Dawidek wrote: >> On Sat, Feb 05, 2011 at 02:36:40PM -0700, Justin T. Gibbs wrote: >>> >>> Perhaps IllumOS will accept these changes back?  As I mentioned in the >>> change descriptions included with the patc

Re: [PATCH] OpenSolaris/ZFS: C++ compatibility

2011-02-05 Thread Justin T. Gibbs
On 2/5/2011 3:06 PM, Pawel Jakub Dawidek wrote: > On Sat, Feb 05, 2011 at 02:36:40PM -0700, Justin T. Gibbs wrote: >> >> Perhaps IllumOS will accept these changes back? As I mentioned in the >> change descriptions included with the patch, the header files already >> show the intention of providing

Re: [PATCH] OpenSolaris/ZFS: C++ compatibility

2011-02-05 Thread Pawel Jakub Dawidek
On Sat, Feb 05, 2011 at 02:36:40PM -0700, Justin T. Gibbs wrote: > On 2/5/2011 8:39 AM, Pawel Jakub Dawidek wrote: > > On Fri, Feb 04, 2011 at 11:03:53AM -0700, Justin T. Gibbs wrote: > >> The attached patch is sufficient to allow a C++ program to use libzfs. > >> The motivation for these changes i

Re: [PATCH] OpenSolaris/ZFS: C++ compatibility

2011-02-05 Thread Justin T. Gibbs
On 2/5/2011 8:39 AM, Pawel Jakub Dawidek wrote: > On Fri, Feb 04, 2011 at 11:03:53AM -0700, Justin T. Gibbs wrote: >> The attached patch is sufficient to allow a C++ program to use libzfs. >> The motivation for these changes is work I'm doing on a ZFS fault >> handling daemon that is written in C++

Re: [PATCH] OpenSolaris/ZFS: C++ compatibility

2011-02-05 Thread Pawel Jakub Dawidek
On Fri, Feb 04, 2011 at 11:03:53AM -0700, Justin T. Gibbs wrote: > The attached patch is sufficient to allow a C++ program to use libzfs. > The motivation for these changes is work I'm doing on a ZFS fault > handling daemon that is written in C++. SpectraLogic's intention > is to return this work