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

Richard Sharpe rsharpe at richardsharpe.com
Fri Apr 22 01:20:34 GMT 2005


On Fri, 22 Apr 2005 mimir at samba.org 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


More information about the samba-technical mailing list