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-cvs/attachments/20050422/3e720e4e/attachment.bin
More information about the samba-cvs
mailing list