[pacman-dev] [patch] resolvedeps cleanup + pactest, alpm_list_add

Xavier shiningxc at gmail.com
Thu Jul 12 15:03:20 EDT 2007


On Thu, Jul 12, 2007 at 08:57:08PM +0200, ngaba at petra.hos.u-szeged.hu wrote:
> > I noticed you added that comment in the code, but I really don't understand
> > why..
> > Either before or after your patch, I don't see why this assumption is needed.
> > Or maybe it isn't needed for that resolvedeps function itself, but for the
> > code which use the result of resolvedeps ?
> 
> Just to be quick: It can be seen from function definition: alpm_list_t 
> *list is used instead of **list.

Hm I see, but isn't that weird ?
Wouldn't it be better to use **list instead then ? Or having resolvedeps
return this list ?




More information about the pacman-dev mailing list