Re: [evolution-patches] e_iconv_* replaced with g_iconv_*



On Mon, 2006-11-13 at 17:08 +0100, Philip Van Hoof wrote:

> This patch replaces the e_iconv_open, e_iconv, e_iconv_close usage in
> Camel with the g_iconv_open, g_iconv and g_iconv_close APIs

It looks like I can also remove the iconv_cache hashtable (or
re-implement the cache using GIConv? Maybe it's interesting to have a
cache like this, it feels like overkill to me, oh well)

Hmm, I'll take a look at the hashtable this evening. 

The ideal situation is that I can remove this iconv detector in the
configure script (as it makes cross compilation a lot more difficult if
I have to compile something just to let it generate some include file)
and if that I can reuse existing functionality from glib (like GIConv).

> This is a tracker bug for upstream Camel:
> http://bugzilla.gnome.org/show_bug.cgi?id=374734
> 
> Tinymail mailing list: this patch will probably soon go in the SVN repo.
> I will first do a few basic regression tests this evening.

-- 
Philip Van Hoof, software developer
home: me at pvanhoof dot be
gnome: pvanhoof at gnome dot org
work: vanhoof at x-tend dot be
blog: http://pvanhoof.be/blog




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]