Re: [Patch] Locate: Move counting and limit-checking into visitors.

[James Youngman]

On Wed, Jun 15, 2005 at 10:10:55PM +0200, Bas van Gompel wrote:
> Hi,
> 
> Following patch moves limit-checking and counting into visitors.
> When neither is done, a visitor is added which will increment
> items_accepted once, then remove itself from the list.

I like visit_count() and visit_limit() but I'm not sure about
visit_found().  The latter is interesting but complicates the code -
what are the benefits that make it worthwhile to make the code more
complex?  I admit that the difference is small.

I'd prefer to adopt visit_count() and visit_limit() but not
visit_found().  However, if you can think of a strong agrument in
favour of visit_found() I could be convinced.  

James.