They both return pointers to internal things. The e-destination.c code frees up the list because e_contact_get_attributes creates a copied GList*.
Also, looking at that e-destination code, there are a few bugs in it too. Chris On Thu, 2004-04-29 at 22:46 +0200, Damien Sandras wrote: > Le jeu, 29/04/2004 � 13:33 -0700, Chris Toshok a �crit : > > neither of those should be called. > > > > Are they pointers to internal things? > > Because in > evolution/addressbook/util/e-destination.c > I see e_vcard_attribute_free is called on the mail list. > > Anyway, my last question about this is should I free each element of the > list returned by e_vcard_attribute_param_get_values? > > Thank you very much for your clarifications! > > > Chris > > > > On Thu, 2004-04-29 at 22:33 +0200, Damien Sandras wrote: > > > And am I supposed to call g_list_foreach (attr_param_list, (GFunc) > > > e_vcard_attribute_param_free, NULL); > > > > > > and g_list_foreach (attr_list, (GFunc) e_vcard_attribute_free, NULL); ? > > > > > > Evolution doesn't seem to call the first one when getting the params > > > list, which could be a leak. However, when I'm calling those functions, > > > I get a crash with an invalid backtrace so I'm a bit puzzled. > > > > > > I'm at the end of my implementation, sorry if I'm harassing you with > > > those mails but I like to be sure of what I'm doing and as all things > > > are not documented yet, I prefer to ask. > > > > > > > > > Thank you very much; > > > > > > > > > Le jeu, 29/04/2004 � 10:30 -0700, Chris Toshok a �crit : > > > > not if you only run the inner loop if the attribute name is "TEL".. > > > > > > > > Chris > > > > > > > > On Thu, 2004-04-29 at 14:52 +0200, Damien Sandras wrote: > > > > > Le jeu, 29/04/2004 � 17:33 +0530, Sivaiah N a �crit : > > > > > > > > > > [...] > > > > > > > > > > > Once you find an attribute having a parameter with name as "TYPE" and > > > > > > value as X-GNOME-MEETING_SPEEDDIAL , you can call > > > > > > e_vcard_attribute_get_value or e_vcard_attribute_get_values on that > > > > > > attribute. Not sure though :) > > > > > > > > > > > > > > > > Well, that's what I'm currently doing, but as I'm playing with lists of > > > > > lists of lists and that the attribute is in the top-level list, but > > > > > X-GNOMEMEETING_SPEEDDIAL in one of the sublist, isn't there a risk to > > > > > get a value for something else that TYPE/X-GNOMEMEETING_SPEEDDIAL ? > > > > > > > _______________________________________________ > > evolution-hackers maillist - [EMAIL PROTECTED] > > http://lists.ximian.com/mailman/listinfo/evolution-hackers _______________________________________________ evolution-hackers maillist - [EMAIL PROTECTED] http://lists.ximian.com/mailman/listinfo/evolution-hackers
