New talloc feature: memlimits

simo idra at samba.org
Fri Oct 5 08:47:25 MDT 2012


On Fri, 2012-10-05 at 10:12 -0400, simo wrote:
> On Fri, 2012-10-05 at 15:36 +0200, David Disseldorp wrote:
> > Hi Simo,
> > 
> > On Sat, 29 Sep 2012 00:45:20 -0400
> > simo <idra at samba.org> wrote:
> > 
> > > You can find it here:
> > > https://git.samba.org/idra/samba.git/?p=idra/samba.git;a=shortlog;h=refs/heads/talloc_memlimit
> > 
> > Could you please avoid the nested returns in the TALLOC_MEMLIMIT_*
> > macros? As discussed previously, macros that affect control flow break
> > the internal parsers of those who read the code[1].
> > 
> > Cheers, David
> > 
> > 1) http://lists.samba.org/archive/samba-technical/2012-May/084115.html
> 
> Ah that is a good point, sorry for doing that myself.
> Expect a fix soon.

Ok I have a patch in autobuild to turn the macros into static functions
and stop returning implicitly.

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer <simo at samba.org>
Principal Software Engineer at Red Hat, Inc. <simo at redhat.com>



More information about the samba-technical mailing list