Looks fine to me.

-Srini
On Tue, 2006-01-10 at 23:31 +0530, Harish Krishnaswamy wrote:
> This change looks fine to me. Actually, it must have been just oversight
> not having removed e-util-marshal.list from evolution/widgets/misc.
>  
> Srini has been working on widgets/* more than any of us lately, though.
> Thoughts, Srini ?
> 
> Thanks,
> Harish
> On Tue, 2006-01-10 at 18:47 +0800, simon.zheng wrote:
> > Hi all,
> > 
> > Here is bug information.
> > http://bugzilla.gnome.org/show_bug.cgi?id=323529
> > 
> > We found another duplicate file "e-util-marshal.list". There's two
> > copies of e-util-marshal.list in and "evo/e-util" and
> > "evo/widgets/misc". They're 100% identical. What's more, we noticed the
> > other modules in evo/widgets, such as evo/widgets/table and
> > evo/widgets/text, use the copy in evo/e-util rather than their own
> > built-in copies. We think the one in evo/widgets/misc might be dropped.
> > 
> > Attached the patch, pls review and comment.
> > 
> > Thanks,
> > -Simon
> > _______________________________________________
> > Evolution-hackers mailing list
> > Evolution-hackers@gnome.org
> > http://mail.gnome.org/mailman/listinfo/evolution-hackers
> 
> _______________________________________________
> Evolution-hackers mailing list
> Evolution-hackers@gnome.org
> http://mail.gnome.org/mailman/listinfo/evolution-hackers

_______________________________________________
Evolution-hackers mailing list
Evolution-hackers@gnome.org
http://mail.gnome.org/mailman/listinfo/evolution-hackers

Reply via email to