On Wed, Feb 23, 2011 at 3:43 AM, Stefan Fuhrmann
<stefanfuhrm...@alice-dsl.de> wrote:
[snip]
> List of integration branches:
> -----------------------------
>
> integrate-is-cachable
> - replace the fixed 1MB limit to the cachable full-text
>  size in FSFS with a cache-specific implementation

Took a look at this one.  The changes look good.  The only bit I
didn't understand was this fragment:

{{{
+/* Store fulltext in RB in the fulltext cache used by said RB. Items that
+ * are too large to be cached won't. Also, this will be a no-op if no
+ * fulltext cache has been enabled in RB.
+ */
 static svn_error_t *
 rep_read_contents_close(void *baton)
}}}

I think that comment is probably misplaced.  The only thing in there
are two destroy functions. :-)

-John

Reply via email to