Re: Merging work
- From: Philip Van Hoof <spam pvanhoof be>
- To: Jose Dapena Paz <jdapena igalia com>
- Cc: tinymail-devel-list <tinymail-devel-list gnome org>
- Subject: Re: Merging work
- Date: Mon, 08 Oct 2007 11:53:13 +0200
Hi there,
These are the two changes that I filtered out of the diff and your
comments, Jose.
http://tinymail.org/trac/tinymail/changeset/2828
Thanks for the review of the changes.
On Mon, 2007-10-08 at 08:15 +0200, Jose Dapena Paz wrote:
> El lun, 08-10-2007 a las 00:15 +0200, Philip Van Hoof escribi� > Hey Jose,
> >
> > Would it be possible for you to check the differences of upstream
> > Camel's camel-mime-utils.c with camel-lite's. I refrained from
> > synchronising our camel-lite's camel-mime-utils.c with upstream Camel
> > because there might have been changes that you made to support the
> > attachment purging. I've found some changes that were unknown to me.
>
> I've been reviewing the svn log, and it seems I didn't make any change
> in camel mime utils (and after reviewing the differences between
> upstream camel and tinymail camel I still don't find anything I can
> remember).
>
> Anyway, briefly, the changes are:
> * A lot of compilation warning fixes, using proper typecasts. They
> should go upstream.
> * The change in parenthesis and conditions in line 795 present in
> upstream camel seems to be interesting for us, as it seems an obvious
> error.
> * Our fix for escaping ? seems ok at lines 1155 and 1168. Should go
> upstream.
> * You changed CamelContentType instantiation to use g slice. Should be
> upstream unless it breaks something there for backwards compatibility.
> * We should get the change at line 2097 upstream. But it seems even
> upstream is not ok, as the *inptr validation should be
> (*inptr && camel_mime_is_dtext(*inptr))
> instead of
> (camel_mime_is_dtext(*inptr) && *inptr)
>
> In general there are no traumatic changes, and they are all more or
> less wise both in our side and on camel upstream side. So I would say we
> should make these changes go upstream in official camel (the only doubt
> would be the GSlice issue).
>
--
Philip Van Hoof, software developer
home: me at pvanhoof dot be
gnome: pvanhoof at gnome dot org
http://www.pvanhoof.be/blog
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]