[Freeciv-Dev] [patch #1651] genlist_list_update() and genlist_list_index()

2010-11-23 Thread Matthias Pfafferodt
Update of patch #1651 (project freeciv): Status: In Progress = Invalid Open/Closed:Open = Closed ___ Follow-up Comment #5: I think this patch

[Freeciv-Dev] [patch #1651] genlist_list_update() and genlist_list_index()

2010-10-25 Thread pepeto
Follow-up Comment #4, patch #1651 (project freeciv): Reading over this item I forgot about... I just posted some extension of the genlist. It makes genlist_list_index() obsolete, as genlist_search() now returns the link. genlist_list_update() maybe should be genlist_link_set(). But, I still

[Freeciv-Dev] [patch #1651] genlist_list_update() and genlist_list_index()

2010-04-25 Thread Matthias Pfafferodt
Follow-up Comment #2, patch #1651 (project freeciv): genlist_list_update() returns the old value; the caller has to free it; I did not knew about hash_replace(). ___ Reply to this item at: http://gna.org/patch/?1651

[Freeciv-Dev] [patch #1651] genlist_list_update() and genlist_list_index()

2010-04-25 Thread pepeto
Follow-up Comment #3, patch #1651 (project freeciv): hash_replace() also returns it, but it frees it before returning, maybe that's a wrong way. But lot of code assume it does... ___ Reply to this item at: http://gna.org/patch/?1651