A small patch for C++ VFS modules.

Jeremy Allison jra at samba.org
Mon Jun 27 16:54:54 GMT 2005


On Sat, Jun 25, 2005 at 05:12:04PM +1000, Ben Martin wrote:
> On Fri, 2005-06-24 at 13:26 -0700, Jeremy Allison wrote:
> > On Fri, Jun 24, 2005 at 11:23:40AM +1000, Ben Martin wrote:
> > > 
> > > OK, I had started doing it this way and reverted to header only. I'm
> > > happy to resubmit with the more intrusive versions. 
> > 
> > You shouldn't need to. In order to make it easier for people to write
> > modules in C++ I've fixed our headers and code to make it C++ friendly.
> > 
> > Checkout SAMBA_3_0 svn and please let me know it I missed anything.
> > 
> > Jeremy.
> 
> Excellent stuff! 
> 
> One minor thing: include/smb_acls.h 
> 
> .../source/include/smb_acls.h:262: error: conflicting declaration 'typedef struct SMB_ACL_T* SMB_ACL_T'
> .../source/include/smb_acls.h:260: error: 'struct SMB_ACL_T' has a previous declaration as 'struct SMB_ACL_T'
> .../source/include/smb_acls.h:266: error: conflicting declaration 'typedef struct SMB_ACL_ENTRY_T* SMB_ACL_ENTRY_T'
> .../source/include/smb_acls.h:264: error: 'struct SMB_ACL_ENTRY_T' has a previous declaration as 'struct SMB_ACL_ENTRY 

Fixed in SVN (by appending '_' to the struct def).

Jeremy.


More information about the samba-technical mailing list