[PATCH 2 of 8] localrepo: rename in-memory tag cache instance attributes

Greg Ward greg-hg at gerg.ca
Mon Jul 13 07:07:42 CDT 2009


On Mon, Jul 13, 2009 at 4:20 AM, timeless<timeless at gmail.com> wrote:
>> +        # These two define the set of tags for this repository.  _tags
>> +        # maps tag name to node;
>
> my initial response was to say you wanted "nodes", but i think for
> proper English it would be "maps a tag name to a node".

Counterclaim: the idiom "maps X to Y" is a peculiar special case used
in math and programming; it appears particularly often in comments and
docstrings of Python or Java code written by me.  ;-)  I'm speaking of
"tag name" in the generic sense, not about any particular tag name or
set of tag names.  Hence, I claim that "maps tag name to node" is
valid mathematician/programmer English.  (And if you don't like
*that*, I'll just play the "descriptive vs prescriptive, language is a
living thing, nyah nah nah" card.  ;-)  (Mostly kidding!  I do
actually believe in objective external reality, even for grammar.)

Your other comments are all fine and I've made the suggested changes.  Thanks!

Greg



More information about the Mercurial-devel mailing list