[SCM] Samba Shared Repository - branch master updated

Jeremy Allison jra at samba.org
Wed May 30 10:52:44 MDT 2012


On Wed, May 30, 2012 at 12:17:59PM +0200, Volker Lendecke wrote:
> On Wed, May 30, 2012 at 11:25:59AM +0200, David Disseldorp wrote:
> > On Wed, 30 May 2012 07:59:22 +0200
> > Volker Lendecke <Volker.Lendecke at SerNet.DE> wrote:
> > 
> > > Doing a return or
> > > goto from something that looks like a function call is just
> > > wrong to me. To save precious screen space, I would rather
> > > go and introduce a special rule to say
> > > 
> > > if (ptr == NULL) { return WERR_NOMEM };
> > > 
> > > in one line. I know we have tons of uses of those, but I
> > > would like to start a discussion about banning them.
> > 
> > +1 from me, macros that effect control flow are evil IMO.
> > 
> > My preference would be to use a two line if statement:
> > if (ptr == NULL)
> > 	 return WERR_NOMEM;
> 
> If nobody objects, I will push the attached patch by the end
> of this week.
> 
> With best regards,

+1 from me. Macros that affect program control flow deserve
a special place in hell :-).


More information about the samba-technical mailing list