[pacman-dev] [PATCH] alpm/handle.c: ensure handle is not NULL before proceeding

Xavier Chantry chantry.xavier at gmail.com
Mon Mar 28 15:28:27 EDT 2011

On Mon, Mar 28, 2011 at 9:26 PM, Rémy Oudompheng
<remyoudompheng at gmail.com> wrote:
> On 2011/3/28 Rémy Oudompheng <remyoudompheng at gmail.com> wrote:
>> Many alpm_option_get/set_*() functions already check this
>> and set pm_errno to the right value, but not all, so
>> this improves consistency.
> I just noticed the ASSERT..RET_ERR construct, should it be the
> preferred way of implementing such things ?

I was going to say a macro would be nice for that repetitive stuff.

And indeed we already have them and there is already that code :

More information about the pacman-dev mailing list