[pacman-dev] Possible redudant condition in handle.c

Helder Martins heldermartins89 at gmail.com
Mon Aug 29 16:37:48 EDT 2011


I saw this little piece of code in handle.c in libpalm, starts at line 130:

	ASSERT(handle->lockfile != NULL, return -1);
     -> ASSERT(handle->lckstream != NULL, return 0);

     -> if(handle->lckstream != NULL) {
		handle->lckstream = NULL;

So i was wondering if that assert or the if condition are redundant, and
if yes which one has the correct behaviour for the objective of this

I can provide the patch once i know which one is the correct behaviour.

Helder Martins

More information about the pacman-dev mailing list