Re: New implementation of EMsgPort

Philip Van Hoof <[email protected]> Thu, 21 Sep 2006 02:18:25 +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].
> 
> 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.

Cool, this is nice stuff Matthew, thanks.

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