From: Sverre Rabbelier <srabbelier@gmail.com>
Date: Sun, 28 Mar 2010 18:13:10 GMT
Subject: Re: [PATCH 0/3] fast textconv
Message-ID: <fabb9a1e1003281113o4cb1e723p4ed39521b645fffe@mail.gmail.com>
In-Reply-To: <20100328173420.GA18047@coredump.intra.peff.net>

Heya,

On Sun, Mar 28, 2010 at 11:34, Jeff King <peff@peff.net> wrote:
> Here is a quick and dirty in-core implementation. The most notable
> defect is that all textconvs store under refs/notes/textconv, which is
> obviously bogus if you might textconv the same blob in two different
> ways.

What I did for the implementation of remote helpers is hash the url of
the remote when storing it locally, that way you don't have to worry
about escaping urls etc. You could do the same with the textconvs,
store it under  refs/notes/textconv/<hash of textconv filter>?

-- 
Cheers,

Sverre Rabbelier