[PATCHES] lib/util - cleanups, build restructuring, charset compatibility

Martin Schwenke martin at meltin.net
Thu Jan 8 11:18:50 MST 2015


On Thu, 08 Jan 2015 10:27:06 +0100, Andreas Schneider <asn at samba.org>
wrote:

> On Thursday 08 January 2015 20:05:22 Martin Schwenke wrote:
> > On Wed, 07 Jan 2015 17:40:26 +0100, Andreas Schneider <asn at samba.org>
> > 
> > wrote:
> > > I think SMB_ASSERT should only be defined if the samba macro is available.
> > > This means debug.h should have #define SAMBA_DEBUG 1
> > > 
> > > #ifdef SAMBA_DEBUG
> > > #define SMB_ASSERT() ...
> > > #endif
> > 
> > debug.h already defines _DEBUG_H.  Perhaps we should change that to
> > _SAMBA_DEBUG_H_ for consistency with other headers and test that in
> > fault.h?
> 
> The attached patch should address the issue.

Reviewed-by: Martin Schwenke <martin at meltin.net>

Pushed to autobuild.  Please re-push if it doesn't make it through...

peace & happiness,
martin


More information about the samba-technical mailing list