Re: Silly coding in pgcrypto

On Sun, Nov 02, 2014 at 05:10:25AM +0100, Marko Tiikkaja wrote:
> *** a/contrib/pgcrypto/pgp-decrypt.c
> --- b/contrib/pgcrypto/pgp-decrypt.c
> ***************
> *** 1069,1075 **** pgp_skip_packet(PullFilter *pkt)
>   
>       while (res > 0)
>           res = pullf_read(pkt, 32 * 1024, &tmp);
> !     return res < 0 ? res : 0;
>   }
>   
>   /*
> --- 1069,1075 ----
>   
>       while (res > 0)
>           res = pullf_read(pkt, 32 * 1024, &tmp);
> !     return res;

Why is the old code silly and the new code correct?