On Fri, 22 Apr 2005 [EMAIL PROTECTED] wrote:
> Author: mimir
> Date: 2005-04-22 00:32:37 +0000 (Fri, 22 Apr 2005)
> New Revision: 6426
[deletia]
> Changeset:
> Modified: branches/SAMBA_4_0/source/torture/libnet/userinfo.c
> ===================================================================
> --- branches/SAMBA_4_0/source/torture/libnet/userinfo.c 2005-04-22
> 00:22:11 UTC (rev 6425)
> +++ branches/SAMBA_4_0/source/torture/libnet/userinfo.c 2005-04-22
> 00:32:37 UTC (rev 6426)
> @@ -225,17 +225,8 @@
> DCERPC_SAMR_UUID,
> DCERPC_SAMR_VERSION);
>
> - 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.
Of course, I am not going to create a big fuss, though.
Regards
-----
Richard Sharpe, rsharpe[at]richardsharpe.com, rsharpe[at]samba.org,
sharpe[at]ethereal.com, http://www.richardsharpe.com
