Rev 5294: merge redundant if-statements. in http://samba.sernet.de/ma/bzr/SAMBA_3_0-registry.bzr/

[Michael Adam]

Hi,

the attached mini-patch to SAMBA_3_0 combines the bodies of two 
if/else blocks removing one redundant if statement. 
Functionality is not changed.

Cheers, Michael

On Di, Mär 20, 2007 at 05:26:26 +0100, Michael Adam wrote:
> At http://samba.sernet.de/ma/bzr/SAMBA_3_0-registry.bzr/
> 
> ------------------------------------------------------------
> revno: 5294
> revision-id: ma at sernet.de-20070320162624-8e43ac647ee3993b
> parent: ma at sernet.de-20070320152704-aaf42e1b9a7b1f4f
> committer: Michael Adam <ma at sernet.de>
> branch nick: SAMBA_3_0-registry.bzr
> timestamp: Tue 2007-03-20 17:26:24 +0100
> message:
>   merge redundant if-statements.
> modified:
>   source/param/loadparm.c        loadparm.c-20060530022627-1efa1edb3eb0e897
> === modified file 'source/param/loadparm.c'
> --- a/source/param/loadparm.c	2007-03-20 05:06:32 +0000
> +++ b/source/param/loadparm.c	2007-03-20 16:26:24 +0000
> @@ -3524,9 +3524,7 @@
>  		parm_ptr =
>  			((char *)ServicePtrs[snum]) + PTR_DIFF(def_ptr,
>  							    &sDefault);
> -	}
>  
> -	if (snum >= 0) {
>  		if (!ServicePtrs[snum]->copymap)
>  			init_copymap(ServicePtrs[snum]);
>  
> 

-- 
Michael Adam <ma at sernet.de>
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: +49-551-370000-0, fax: +49-551-370000-9
AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
http://www.SerNet.DE, mailto: Info @ SerNet.DE
-------------- next part --------------
A non-text attachment was scrubbed...
Name: patch
Type: text/x-patch
Size: 365 bytes
Desc: not available
Url : http://lists.samba.org/archive/samba-technical/attachments/20070320/73c65b28/patch.bin