[PATCH 2 of 3 RESEND] revset: make desc() function accept multiple arguments

[Alexander Plavin]

01.09.2013, 09:48, "Kevin Bullock" <kbullock+mercurial at ringworld.org>:
> On 29 Aug 2013, at 2:33 AM, Alexander Plavin wrote:
>
>>  # HG changeset patch
>>  # User Alexander Plavin <alexander at plav.in>
>>  # Date 1377196193 -14400
>>  #      Thu Aug 22 22:29:53 2013 +0400
>>  # Node ID 52606445710a602c9ea290835f98eb436e78cccd
>>  # Parent  e750d4c05d45410bfd51848374e9b2c479a8235a
>>  revset: make desc() function accept multiple arguments
>>
>>  This provides a more convenient way to express a conjunction of
>>  multiple desc() calls with single argument.
>>
>>  diff -r e750d4c05d45 -r 52606445710a mercurial/revset.py
>>  --- a/mercurial/revset.py Tue Aug 06 00:52:06 2013 +0400
>>  +++ b/mercurial/revset.py Thu Aug 22 22:29:53 2013 +0400
>>  @@ -577,15 +577,20 @@
>>      return [r for r in subset if dm(repo[r].date()[0])]
>>
>>  def desc(repo, subset, x):
>>  -    """``desc(string)``
>>  -    Search commit message for string. The match is case-insensitive.
>>  +    """``desc(*string)``
>
> Existing style would dictate that this should be:
>
> ``desc(string[, string...])``
>
> Likewise for the keyword() description in patch 3.

Are you sure? Existing help contains "ancestor(*changeset)" (but "sort(set[, [-]key...])" :) ).

>
> pacem in terris / мир / शान्ति / ‎‫سَلاَم‬ / 平和
> Kevin R. Bullock