Re: [evolution-patches] Re: (groupwise)fix for bug 72302 - deleted emails not syncing up

[Parthasarathi Susarla <sparthasarathi novell com>]

Committed.
Thanks,
partha
On Fri, 2005-04-08 at 14:26 +0530, Harish Krishnaswamy wrote:
> Looks good. I noticed you have cleaned up some of the code, not aligning
> to the conventions, as well. Great :-)
> 
> Thanks,
> Harish
> 
> On Thu, 2005-04-07 at 19:04 +0530, Parthasarathi Susarla wrote:
> > Please ignore the previous mail, i attached the older patch by mistake.
> > The attached file is the modified one.
> > Thanks
> > 
> > On Thu, 2005-04-07 at 18:59 +0530, Parthasarathi Susarla wrote:
> > > Hi,
> > > Find the modified patch attached.It has the following changes
> > > 	1. Frees all the memory used
> > > 	2. Use GSList instead of GList 
> > > Thanks,
> > > partha
> > > 
> > > On Thu, 2005-04-07 at 15:59 +0530, Harish Krishnaswamy wrote:
> > > > hi,
> > > > 
> > > >   The patch looks fine. You may want to look at the following points
> > > > before committing, though. 
> > > > >        status = e_gw_connection_get_quick_messages (cnc, container_id,
> > > > > "id",
> > > > > +                               &t_str, "All", NULL, NULL, -1,
> > > > > &slist) ;
> > > > 
> > > > * slist and its contents are being leaked here.
> > > > 
> > > > * You are converting slist from a GSList to GList while passing it to
> > > > your internal function - gw_update_all_items. I think you can avoid this
> > > > by using GSList as it is. I do not see what we may gain from this
> > > > conversion.
> > > > 
> > > > * The comment  appears to be wrong. 
> > > > > /*item_list : List of ids from the server*/
> > > > item_list stores the ids in the summary. temp_list stores the ids from
> > > > the summary.
> > > > 
> > > > -Harish
> > > > _______________________________________________
> > > > evolution-patches mailing list
> > > > evolution-patches lists ximian com
> > > > http://lists.ximian.com/mailman/listinfo/evolution-patches
> _______________________________________________
> evolution-patches mailing list
> evolution-patches lists ximian com
> http://lists.ximian.com/mailman/listinfo/evolution-patches