Re: [gamin] gamin-fsset-control.patch



On Fri, Aug 05, 2005 at 02:10:55PM -0400, John McCutchan wrote:
> Yo,
> 
> Daniel, could you review this patch? It changes the code that decides
> when to enable kernel monitoring / polling. It follows what we discussed
> earlier: The exclude list has higher authority than the fsset code.
> 
> I think it's correct, but it needs to be reviewed before it gets
> committed.

  That looks correct. It may be nicer to factorize the 2 functions
call to get more readable test conditions but it's cosmetic :-)

Daniel

-- 
Daniel Veillard      | Red Hat Desktop team http://redhat.com/
veillard redhat com  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/



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