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

[Helder Martins]

Hi.

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) {
                fclose(handle->lckstream);
                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
function.

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

Thanks,
Helder Martins