Re: New implementation of EMsgPort

Philip Van Hoof <[email protected]> Tue, 17 Oct 2006 09:59:24 +0200
Newsgroups gmane.comp.gnome.evolution.patches
Message-ID <[email protected]>
On Wed, 2006-09-20 at 13:17 -0400, Matthew Barnes wrote:
> I would appreciate a review of my patch for bug #348888 [1].

This implementation has been merged with the camel-lite which is going
to be used by tinymail. This adds another tester for your patch.

Thanks.

> The patch is a new implementation of EMsgPort using a GAsyncQueue.  It
> also restores on-demand creation of pipes to keep file descriptor usage
> from spiraling out of control at start-up, but without the race
> conditions that forced the feature to be yanked earlier this year.  It
> also keeps the existing API intact.
> 
> The new implementation seems pretty stable.  I've been running with it
> for about a week now and haven't experienced any deadlocks or crashes.
> 
> Obviously we should wait until CVS is branched before committing it.


-- 
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