Re: [evolution-patches] New implementation of EMsgPort



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




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