[PATCH 1 of 2] template: add shortestnode keyword

[Durham Goode]

On 1/17/14 6:20 AM, "Augie Fackler" <raf at durin42.com> wrote:

>On Fri, Jan 17, 2014 at 12:45:29AM -0800, Durham Goode wrote:
>> # HG changeset patch
>> # User Durham Goode <durham at fb.com>
>> # Date 1389946237 28800
>> #      Fri Jan 17 00:10:37 2014 -0800
>> # Node ID b18359a70b640d2aeb3f4afd1c8e47d775402b8e
>> # Parent  6545770bd37991b4ff0400479455a6e3ffa5976b
>> template: add shortestnode keyword
>
>I see what, but not a why. What's the motivation for having some sort
>of shortnode less than 12 bytes?

Because it's easier to read and type.  In particular, on small repos (like
the mercurial repo), it ends up being 4 characters, which I can fit in my
human memory for typing.  Rev numbers don't work for this because there's
a lot of repetition in the first few digits, making it hard to remember
for typing.

It's the difference between my log looking like this:

@  durham   shortestnode
|  77cf62dd2af0 template: add pad function for padding output
|
o  durham
|  b18359a70b64 template: add shortestnode keyword
|
o  pierre-yves @
|  6545770bd379 backout: add a message after backout that need ...
|
| o  durham   manifestcache
|/   93f07171bf64 manifest cache
|
| o  durham   catperf
| |  c7652078e0d4 cat: increase perf when catting single files
| |
| o  durham
|/   9c531e73a03e changectx: increase perf of walk function
|



And like this:

@  durham  shortestnode
|  77cf    template: add pad function for padding output
|
o  durham
|  b183    template: add shortestnode keyword
|
o  pierre-yves @
|  6545    backout: add a message after backout that need manual commit
|
| o  durham   manifestcache
|/   93f0    manifest cache
|
| o  durham   catperf
| |  c765    cat: increase perf when catting single files
| |
| o  durham
|/   9c53    changectx: increase perf of walk function
|