bryson/pacman
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Add a little const correctness fix to alpm_list

Browse source 
Signed-off-by: Dan McGee <dan@archlinux.org>
This commit is contained in:
Dan McGee 2007-10-31 11:37:49 -05:00
parent dea9b3bc0f
commit ab91cef655
1 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
lib/libalpm/alpm_list.c
View file

@ -346,7 +346,7 @@ alpm_list_t SYMEXPORT *alpm_list_remove_node(alpm_list_t *node)
* @return a new list containing non-duplicate items * @return a new list containing non-duplicate items
*/ */
alpm_list_t SYMEXPORT *alpm_list_remove_dupes(const alpm_list_t *list) alpm_list_t SYMEXPORT *alpm_list_remove_dupes(const alpm_list_t *list)
{ /* TODO does removing the strdup here cause invalid free's anywhere? */ {
const alpm_list_t *lp = list; const alpm_list_t *lp = list;
alpm_list_t *newlist = NULL; alpm_list_t *newlist = NULL;
while(lp) { while(lp) {
@ -426,8 +426,9 @@ alpm_list_t SYMEXPORT *alpm_list_copy_data(const alpm_list_t *list)
* @return a new list in reverse order * @return a new list in reverse order
*/ */
alpm_list_t SYMEXPORT *alpm_list_reverse(alpm_list_t *list) alpm_list_t SYMEXPORT *alpm_list_reverse(alpm_list_t *list)
{ /* TODO any invalid free's from NOT duplicating data here? */ {
alpm_list_t *lp, *newlist = NULL; const alpm_list_t *lp;
alpm_list_t *newlist = NULL;
lp = alpm_list_last(list); lp = alpm_list_last(list);
while(lp) { while(lp) {