svn commit: samba r6426 - in branches/SAMBA_4_0/source/torture/libnet: .

Rafal Szczesniak mimir at samba.org
Fri Apr 22 02:30:06 GMT 2005


On Thu, Apr 21, 2005 at 06:20:34PM -0700, Richard Sharpe wrote:
> On Fri, 22 Apr 2005 mimir at samba.org wrote:
> 
> > -	if (!NT_STATUS_IS_OK(status)) {
> > -		return False;
> > -	}
> > +	if (!NT_STATUS_IS_OK(status)) return False;
> 
> I don't think this is a big item, but I actually prefer the first one
> above that was deleted, because it makes things much easier when you are
> inserting debugging statements and so forth, at the cost of a few more
> characters.

Certainly, when you add debug statements you use the deleted form.
I changed it to single line because it is smaller. Once I put there
something more it will be just what you prefer.


cheers,
-- 
Rafal Szczesniak
Samba Team member  http://www.samba.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
Url : http://lists.samba.org/archive/samba-technical/attachments/20050422/3e720e4e/attachment.bin


More information about the samba-technical mailing list