[PATCH 4 of 4] hgweb: remove now unnecessary explicit header() and footer()

Martin Geisler martin at geisler.net
Fri Aug 9 07:30:05 CDT 2013


Alexander Plavin <alexander at plav.in> writes:

> 09.08.2013, 15:30, "Martin Geisler" <martin at geisler.net>:
>> Alexander Plavin <alexander at plav.in> writes:
>>
>>>  09.08.2013, 11:07, "Martin Geisler" <martin at geisler.net>:
>>>>  Alexander Plavin <alexander at plav.in> writes:
>>>>>   # HG changeset patch
>>>>>   # User Alexander Plavin <alexander at plav.in>
>>>>>   # Date 1374621626 -14400
>>>>>   #      Wed Jul 24 03:20:26 2013 +0400
>>>>>   # Node ID 11ac049356d4894642313803d9edc1ff2e9d2788
>>>>>   # Parent  1fe956e499f2996d4b63ae04c2ec99fe39e89302
>>>>>   hgweb: remove now unnecessary explicit header() and footer()
>>>>  I think it could be nice if you added a tiny bit of context: when and
>>>>  why did this become unnecessary (which changeset does "now" refer to)?
>>>>
>>>>  My first guess would be that the immediate parent (patch 3) makes the
>>>>  functions unnecessary, but it doesn't look like it to me. So my second
>>>>  guess is that "now" refers to patch 2. That suggests that the patches
>>>>  could be reordered by swapping 3 and 4.
>>>  You are right that this depends on patch 2 only, but why reorder?
>>>  Third patch also depends on that, so it could cause the same question
>>>  as well if they were reordered :)
>>
>> Aha... :-) I somehow thought that moving a string as you do in patch 3
>> was already supported. Maybe you could explain more clearly in patch 2
>> that it allows you to use the map file as a key-value store for plain
>> strings as well (if that is actually what patch 2 does).
>
> I've thought a bit about better commit message for 2nd patch now, but
> couldn't improve what's already there :) Quote from that commit:
> "allows adding arbitrarily-named entries to a template map file, and
> then referencing them" - this includes plain strings too. Any
> suggestion?

I guess the problem is that I don't know what an "arbitrarily-named
entry" is :)

It's been a long time since I looked at the map file format in detail,
but I were asked to describe it, I would describe the format as a bunch of

  key = value

lines where "value" can reference other keys. If I would have to name
one of sides "entry", I would have described the lines this:

  entry = expansion

I now get the impression that you would describe the format as:

  reference = entry

or something similar.

-- 
Martin Geisler


More information about the Mercurial-devel mailing list