Re: Keywords-as-hash final (I hope) version

Max Bowsher wrote:
> Here is what I hope to be the final version of the actual API change
> portion of the keywords-as-hash patch.

Looks good to me; it definitely benefitted from a more thorough rewrite.

> I have removed the implemetation of a $UUID$ keyword from this patch,
> because it is an unrelated feature, and I don't recall any discussion on
> whether $UUID$ was a suitable keyword name.

That's completely fair. The UUID case was requested and I implemented it mostly
to show how easy it was to extend this code to support other keywords.
Personally, I think the UUID is an internal repository concept which doesn't
provide much in the way of useful information to carbon-based readers. The
repos-url, on the other hand, would by much more useful to be exposed to humans
(different problem ;-).

> Based on a patch by John Peacock <jpeacock@rowman.com>.

Truth in advertising - the patch I submitted was closely based on plasma's
(plasmaball@pchome.com.tw) original patch,

        http://www.contactor.se/~dast/svn/archive-2003-05/1975.shtml

brought up to date with the core code changes that occurred over time. The only
code I wrote myself was the compatibility layer and the Python tests (which are
not part of this patch).

John

-- 
John Peacock
Director of Information Research and Technology
Rowman & Littlefield Publishing Group
4720 Boston Way
Lanham, MD 20706
301-459-3366 x.5010
fax 301-429-5747
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org