Sebastian Nowicki wrote:
> I noticed that alpm_list_last simply returns a pointer to the previous  
> node. Is this expected behavior? My interpretation of "last" would be  
> to return the absolute last node, as the comment suggests. In my  
> opinion the function as it is now should be called alpm_list_prev.
> 
>  From alpm_list.c:
>> alpm_list_t SYMEXPORT *alpm_list_last(const alpm_list_t *list)
>> {
>>      if(list) {
>>              return(list->prev);
>>      } else {
>>              return(NULL);
>>      }
>> }
> 

That's because a alpm_list_t element is not really a list, but a list node.
So when you use a list in alpm (with alpm_list_t *list), it's actually 
just the head node that you manipulate.
And head->prev == tail.

_______________________________________________
pacman-dev mailing list
[email protected]
http://archlinux.org/mailman/listinfo/pacman-dev

Reply via email to