On Sun, Aug 10, 2008 at 6:53 PM, Dan McGee <[EMAIL PROTECTED]> wrote:
>
> Note the difference here. Our function doesn't have the ... in the arg
> list- we are already passing in a va_list parameter, so the va_start
> was done elsewhere which gave us the va_list to work on.
>

Oh I see, and alpm_logaction does just that, for this _alpm_logaction function.
So that patch looks perfect.

Hm, just the commit log that I find a bit confusing :
va_list usage is not allowed

Shouldnt it be like in the code comment :
va_list can't be used more than once
_______________________________________________
pacman-dev mailing list
[email protected]
http://archlinux.org/mailman/listinfo/pacman-dev

Reply via email to