[SCM] Samba Shared Repository - branch master updated

simo idra at samba.org
Wed May 30 05:49:36 MDT 2012


On Wed, 2012-05-30 at 12:17 +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.

ACK

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